Skip to content

Height calculation improvements #1536

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 14 commits into from
Feb 19, 2025
Merged

Height calculation improvements #1536

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
34c252c
match lazy comp's loading skeleton height as per comp's default height
raheeliftikhar5 Feb 13, 2025
9d3f8b9
fixed tooltip warnings after antd upgrade
raheeliftikhar5 Feb 13, 2025
4fa9599
memoize timer comp's view and propertyView
raheeliftikhar5 Feb 13, 2025
2311310
memoize transfer comp's view and propertyView
raheeliftikhar5 Feb 13, 2025
ad3733a
reduce re-rendering by adding memoziation
raheeliftikhar5 Feb 13, 2025
2ee3c48
reduced actions for global variables
raheeliftikhar5 Feb 19, 2025
cf5f7a9
fixed console errors from avatar comp
raheeliftikhar5 Feb 19, 2025
c5734f8
reduce delay in height calculation
raheeliftikhar5 Feb 19, 2025
71b4054
remove lazy loading from comps
raheeliftikhar5 Feb 19, 2025
2d2ce6a
optimise bundle size for lowcoder
raheeliftikhar5 Feb 19, 2025
75a84b0
remove console logs
raheeliftikhar5 Feb 19, 2025
1fb1131
Merge branch 'dev' into height-calculation-improvements
FalkWolsky Feb 19, 2025
76c447d
fix build
raheeliftikhar5 Feb 19, 2025
e822d17
fix build
raheeliftikhar5 Feb 19, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove console logs
  • Loading branch information
@raheeliftikhar5
raheeliftikhar5 committed Feb 19, 2025
commit 75a84b083a44ef82fde22c887cad1b672955dd1e
4 changes: 0 additions & 4 deletions client/packages/lowcoder/src/layout/layoutOpUtils.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ export namespace layoutOpUtils {
}

function reduce(layout: Layout, op: LayoutOp, stickyItemMap?: Record<string, Set<string>>): Layout {
// console.log(op.type);
let newLayout = layout;
switch (op.type) {
case LayoutOpTypes.CHANGE_ITEM: {
Expand Down Expand Up @@ -147,7 +146,6 @@ export let getUILayout = (
ops: LayoutOps | undefined,
setHiddenCompHeightZero: boolean = false
): Layout => {
// console.log('layout->before', changedHs, layout);
// log.log("getUILayout. layout: ", layout, " extraLayout: ", extraLayout, " changedHs: ", changedHs, " ops: ", ops);
const stickyItemMap = getStickyItemMap(layout);
const hiddenItemHeight = _.fromPairs(
Expand All @@ -165,9 +163,7 @@ export let getUILayout = (
realOps.forEach((op) => {
layout = reduce(layout, op, stickyItemMap);
});
// console.log('layout->after', layout);
layout = cascade(layout);
// console.log('layout->after->2', layout);
if (!setHiddenCompHeightZero) {
const recoverHiddenOps = _.toPairs(hiddenItemHeight).map(([i, h]) => changeItemOp(i, { h }));
recoverHiddenOps.forEach((op) => {
Expand Down
Loading