Skip to content

ECHOES-1455 Allow scroll in left panel of FilterDropdown at 400px - #760

Open
kimphanh wants to merge 1 commit into
mainfrom
kim/filter-max-height/ECHOES-1455
Open

ECHOES-1455 Allow scroll in left panel of FilterDropdown at 400px#760
kimphanh wants to merge 1 commit into
mainfrom
kim/filter-max-height/ECHOES-1455

Conversation

@kimphanh

@kimphanh kimphanh commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Use in my dev branch

Screenshot 2026-08-28 at 11 47 37

Part of


Summary by Gitar

  • UI styling updates:
    • Updated maxHeight default value to 400px in base design tokens

This will update automatically on new commits.

@netlify

netlify Bot commented Aug 28, 2026

Copy link
Copy Markdown

Deploy Preview for echoes-react ready!

Name Link
🔨 Latest commit 9b23db2
🔍 Latest deploy log https://app.netlify.com/projects/echoes-react/deploys/6a9152e602dce10008cf72dd
😎 Deploy Preview https://deploy-preview-760--echoes-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Aug 28, 2026

Copy link
Copy Markdown

ECHOES-1455

Comment thread src/generated/design-tokens-base.json
gitar-bot[bot]

This comment was marked as resolved.

@kimphanh
kimphanh force-pushed the kim/filter-max-height/ECHOES-1455 branch from 6a86356 to 45cb820 Compare August 28, 2026 09:10
@kimphanh
kimphanh force-pushed the kim/filter-max-height/ECHOES-1455 branch from 45cb820 to 9b23db2 Compare August 28, 2026 09:20
@gitar-bot
gitar-bot Bot dismissed their stale review August 28, 2026 09:21

✅ All code review findings resolved.

Configure merge blocking

@gitar-bot

gitar-bot Bot commented Aug 28, 2026

Copy link
Copy Markdown
Code Review ✅ Approved 1 resolved / 1 findings

Updates the FilterDropdown left panel maxHeight default value to 400px in the design tokens, enabling scroll behavior at smaller viewport sizes. No issues found.

✅ 1 resolved
Bug: Generated token file hand-edited; source token still 600px

📄 src/generated/design-tokens-base.json:106 📄 design-tokens/tokens/component/base.json:40-45
src/generated/design-tokens-base.json is build output of yarn build-tokens (design-tokens/build.js) from design-tokens/tokens/**, where echoes.filter.dropdown.sizes.maxHeight.default is still 600px (design-tokens/tokens/component/base.json:40-45). Three consequences follow from editing only the generated JSON: (1) the change has no runtime effect — FilterDropdownStyles.ts:38 uses cssVar(...), which emits var(--echoes-filter-dropdown-sizes-max-height-default) resolved from src/generated/design-tokens-base.css:128, still 600px, so the dropdown never clamps to 400px and the left panel still does not scroll as the PR intends; (2) CI fails — yarn validate-citokens-checkconfig/scripts/validate-tokens.sh rebuilds tokens and runs git diff --quiet src/generated, which now reports a diff because the regenerated JSON reverts to 600px; (3) the JSON and CSS artifacts disagree, so designToken('filter-dropdown-sizes-max-height-default') (src/utils/design-tokens.ts:81) returns 400px while the applied CSS is 600px. Fix by changing the source token and regenerating, leaving src/generated/ untouched by hand.

Implementation Status ✅ 1 of 1 objectives covered
ECHOES-1455 - 1 of 1 objectives covered

This PR changes the echoes-filter-dropdown-sizes-max-height-default design token value from 600px to 400px.

✅ 1 covered here
Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqube-next

Copy link
Copy Markdown

@kimphanh
kimphanh marked this pull request as ready for review August 28, 2026 09:38
@kimphanh
kimphanh requested a review from gregaubert August 28, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant