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

fix!: Renamed the blocklyTreeSelected CSS class to blocklyToolboxSelected #8459

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

omwagh28
Copy link

@omwagh28 omwagh28 commented Aug 4, 2024

Issue -> #8351

fixes #8351

Description

This pull request updates the CSS class names associated with the selection state of toolbox categories in Blockly. The changes improve clarity and consistency by aligning the CSS class naming with the context in which they are used.

Changes Made

  • Renamed CSS Classes: Updated the CSS class name from blocklyTreeSelected to blocklyToolboxSelected across the relevant files. This change reflects the role of the class in styling the selected state of toolbox categories rather than tree elements.

  • CSS Style Updates: Adjusted the CSS selectors to use the new class name, ensuring that all styling for selected toolbox categories remains functional.

Motivation

The renaming of CSS classes enhances code readability and maintainability by making it clear which elements are being targeted for styling. It eliminates ambiguity and aligns with the conventions used throughout the codebase for naming CSS classes associated with toolbox components.

@omwagh28 omwagh28 requested a review from a team as a code owner August 4, 2024 06:46
@omwagh28 omwagh28 requested a review from cpcallen August 4, 2024 06:46
@github-actions github-actions bot added the PR: fix Fixes a bug label Aug 4, 2024
@omwagh28
Copy link
Author

omwagh28 commented Aug 4, 2024

@cpcallen @BeksOmega Please review this PR

@cpcallen cpcallen requested review from BeksOmega and removed request for cpcallen August 5, 2024 18:32
@cpcallen cpcallen removed their assignment Aug 5, 2024
Copy link
Collaborator

@BeksOmega BeksOmega left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great thank you for the fix @omwagh28 =) Once CI passes I'll get this merged!

@omwagh28
Copy link
Author

omwagh28 commented Aug 5, 2024

Thanks @BeksOmega

@BeksOmega BeksOmega changed the title fix: Renamed the blocklyTreeSelected CSS class to blocklyToolboxSelected fix!: Renamed the blocklyTreeSelected CSS class to blocklyToolboxSelected Aug 6, 2024
@github-actions github-actions bot added breaking change Used to mark a PR or issue that changes our public APIs. PR: fix Fixes a bug and removed PR: fix Fixes a bug labels Aug 6, 2024
@BeksOmega BeksOmega merged commit f10c3b0 into google:rc/v12.0.0 Aug 6, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change Used to mark a PR or issue that changes our public APIs. PR: fix Fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants