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

2015 NDC tracker feedback - Chart sorting switch to be clickable in a more user friendly way #1703

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

SARodrigues
Copy link
Collaborator

No description provided.

Copy link

what-the-diff bot commented Oct 23, 2024

PR Summary

  • Component Modification: The Switch component used for sorting tasks in a particular programming file was replaced with a more efficient CheckInput component. This results in better sorting functionality.

  • Standardized Sorting: The addition of a new standardized format COUNTRY_SORTING for sorting country options allows for uniform sorting operations, significantly reducing inconsistencies across the platform.

  • State Initialization Update: Initial settings (or 'state') that used to organize the data (likely country data) are now updated to use COUNTRY_SORTING.emissions, indicating a new default sort order based on emissions.

  • UI Refactor: The switching toggles (buttons used to switch between different options) interface was renovated, with better structure and visuals. The related code terminology was also updated for better readability - switchTitle was renamed to toggleTitle.

  • Unnecessary Styles Removed: To make the overall style of the platform more unified and clear, unneeded styles related to the old Switch component were removed from the styles.scss file.

  • Added New Style File: A new style file toggle-checkbox.scss has been included. This is specifically for defining the look of the new toggle checkbox feature, which enhances the user interface.

@SARodrigues SARodrigues merged commit b13266b into develop Oct 23, 2024
2 of 3 checks passed
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