Skip to content
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.

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

Icons for grid autoflow dropdown #6214

Merged
merged 5 commits into from
Aug 9, 2024
Merged

Icons for grid autoflow dropdown #6214

merged 5 commits into from
Aug 9, 2024

Conversation

ruggi
Copy link
Contributor

@ruggi ruggi commented Aug 8, 2024

Problem:

The grid autoflow dropdown should have meaningful icons.

Fix:

  • Add the icons to the dropdown select options
  • Reorder the values to match intended designs
  • Add a wrapper for radix-ui's select component in order to be able to use separators/placeholders/etc.
Screen.Recording.2024-08-08.at.19.01.22.mov

Fixes #6213

Copy link
Contributor

github-actions bot commented Aug 8, 2024

Try me

Copy link

relativeci bot commented Aug 8, 2024

#13785 Bundle Size — 62.63MiB (+0.13%).

a1d41dc(current) vs 6339ece master#13783(baseline)

Important

Bundle introduced 1 duplicate package – View changed duplicate packages

Warning

Bundle introduced 3 new packages: @radix-ui/react-select, @radix-ui/number, @radix-ui/react-use-previous – View changed packages

Bundle metrics  Change 6 changes Regression 3 regressions
                 Current
#13785
     Baseline
#13783
Regression  Initial JS 45.73MiB(+0.18%) 45.65MiB
No change  Initial CSS 0B 0B
Change  Cache Invalidation 23.78% 22.15%
No change  Chunks 30 30
No change  Assets 33 33
Change  Modules 4384(+0.09%) 4380
No change  Duplicate Modules 523 523
Change  Duplicate Code 31.59%(-0.09%) 31.62%
Regression  Packages 473(+0.85%) 469
Regression  Duplicate Packages 71(+1.43%) 70
Bundle size by type  Change 2 changes Regression 1 regression Improvement 1 improvement
                 Current
#13785
     Baseline
#13783
Regression  JS 62.62MiB (+0.13%) 62.54MiB
Improvement  HTML 11.16KiB (-0.34%) 11.2KiB

Bundle analysis reportBranch feat/grid-autoflow-iconsProject dashboard


Generated by RelativeCIDocumentationReport issue

@ruggi ruggi merged commit 865c30d into master Aug 9, 2024
13 checks passed
@ruggi ruggi deleted the feat/grid-autoflow-icons branch August 9, 2024 10:47
liady pushed a commit that referenced this pull request Dec 13, 2024
**Problem:**

The grid autoflow dropdown should have meaningful icons.

**Fix:**

- Add the icons to the dropdown select options
- Reorder the values to match intended designs
- Add a wrapper for radix-ui's select component in order to be able to
use separators/placeholders/etc.



https://github.com/user-attachments/assets/7f9ac9c2-9610-4ea9-ba98-a76f77976698



Fixes #6213
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.

icons for autoflow grid values
3 participants