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

Improve notification layout #997

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Improve notification layout #997

wants to merge 1 commit into from

Conversation

boatbomber
Copy link
Member

Changes:

Before & After:
image
image

  • The text wrapping no longer shifts around as the notification animates in.

Before & After: (Slowed to show the effect more clearly)

before.mp4
after.mp4

@boatbomber boatbomber requested a review from Dekkonot December 24, 2024 08:50
@boatbomber boatbomber added scope: plugin Relevant to the Roblox Studio plugin regression This bug wasn't a bug in a previous version of Rojo size: small impact: small Minor papercuts in Rojo that don't warrant immediate resolutoin. status: needs review This work is mostly done, but just needs work to integrate and merge it. skip changelog PRs that may skip the changelog enforcement check labels Dec 24, 2024
@boatbomber boatbomber enabled auto-merge (squash) December 24, 2024 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dekkonot Dekkonot Awaiting requested review from Dekkonot

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
impact: small Minor papercuts in Rojo that don't warrant immediate resolutoin. regression This bug wasn't a bug in a previous version of Rojo scope: plugin Relevant to the Roblox Studio plugin size: small skip changelog PRs that may skip the changelog enforcement check status: needs review This work is mostly done, but just needs work to integrate and merge it.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@boatbomber