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.

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

Jump to bottom

Table | Default sort should not consider lower/uppercase when sorting strings #479

Closed
aVileBroker opened this issue Oct 16, 2023 · 1 comment · Fixed by #489
Closed

Table | Default sort should not consider lower/uppercase when sorting strings #479

aVileBroker opened this issue Oct 16, 2023 · 1 comment · Fixed by #489
Labels
0.5 Story Points issue size: .5 bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed released

Comments

@aVileBroker
Copy link
Contributor

Currently, lowercase strings are considered "lower" in a sorted list than other uppercase strings

Sorting should not consider capitalization.

Dev hint: One option is to set the strings to lower-case when sorting them
@aVileBroker aVileBroker added bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed 0.5 Story Points issue size: .5 labels Oct 16, 2023
@aVileBroker aVileBroker mentioned this issue Nov 15, 2023
Merged
aVileBroker added a commit to aVileBroker/foundry-ui that referenced this issue Nov 15, 2023
@aVileBroker
fix(table): default table sorting now correctly ignores capitalized l…
de312b9 
…etters in sort

Previously, the table would put all capitalized words ahead of lowercased words

Fix Headstorm#479
aVileBroker added a commit that referenced this issue Nov 15, 2023
@aVileBroker
fix(table): default table sorting now correctly ignores capitalized l…
1069d90 
…etters in sort (#489)

Previously, the table would put all capitalized words ahead of
lowercased words

Fix #479
HeadstormOps pushed a commit that referenced this issue Nov 15, 2023
@semantic-release-bot
chore(release): 1.27.6 [skip ci]nn## [1.27.6](v1.27.5...v1.27.6) (202…
041ec69 
…3-11-15)

### Bug Fixes

* **table:** default table sorting now correctly ignores capitalized letters in sort ([de312b9](de312b9)), closes [#479](#479)
* **table:** default table sorting now correctly ignores capitalized letters in sort ([#489](#489)) ([1069d90](1069d90)), closes [#479](#479)
@HeadstormOps
Copy link

🎉 This issue has been resolved in version 1.27.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
0.5 Story Points issue size: .5 bug Something isn't working good first issue Good for newcomers help wanted Extra attention is needed released
Projects
Foundry-React-UI
Status: Done
Milestone
No milestone
2 participants
@aVileBroker @HeadstormOps