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

feat: implement WorkspaceSvg class manipulation #8473

Merged

Conversation

UCYT5040
Copy link
Contributor

@UCYT5040 UCYT5040 commented Aug 8, 2024

The basics

The details

Resolves

Fixes #8281

Proposed Changes

Implements addClass and removeClass methods to the WorkspaceSvg class.

Reason for Changes

As requested in #8281

Test Coverage

Ran the usual build/test scripts and also made sure it worked okay in the playground.

Documentation

Yes, add docs for the new methods

Additional Information

@UCYT5040 UCYT5040 requested a review from a team as a code owner August 8, 2024 21:19
Copy link

conventional-commit-lint-gcf bot commented Aug 8, 2024

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@github-actions github-actions bot added the PR: feature Adds a feature label Aug 8, 2024
@BeksOmega BeksOmega requested review from BeksOmega and removed request for rachel-fenichel August 12, 2024 16:44
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.

Looks great! Just two little docs fixes then I can get this merged =)

core/workspace_svg.ts Outdated Show resolved Hide resolved
core/workspace_svg.ts Outdated Show resolved Hide resolved
UCYT5040 and others added 2 commits August 12, 2024 15:34
Co-authored-by: Beka Westberg <[email protected]>
Co-authored-by: Beka Westberg <[email protected]>
@UCYT5040
Copy link
Contributor Author

I accepted both of those. Sorry about those commit names- I didn't notice that it was using the default name when accepting the suggestion.

@BeksOmega
Copy link
Collaborator

No worries at all! Thank you for your work on this =)

@BeksOmega BeksOmega merged commit 731fb40 into google:rc/v12.0.0 Aug 13, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: feature Adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants