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

Enhance backdrop modal effect #652

Merged
merged 2 commits into from
Oct 23, 2023
Merged

Enhance backdrop modal effect #652

merged 2 commits into from
Oct 23, 2023

Conversation

rasvanjaya21
Copy link
Contributor

  • What kind of change does this PR introduce?
    Enhancing, backdrop color effect in when trigger upload new document modal and trigger upgrade modal both using tailwind utility class named gray-alpha enhanced the opacity from .1 to .64
  • Why was this change needed? (You can also link to an open issue here)
    Design Consistency, refer to when trigger sign up or sign in modal their backdrop color is darker than other modals (the difference is in the opacity value)
  • Other information:

@vercel
Copy link

vercel bot commented Oct 19, 2023

@rasvanjaya21 is attempting to deploy a commit to the Arc53 Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 23, 2023 4:22pm
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 23, 2023 4:22pm

@rasvanjaya21
Copy link
Contributor Author

reference

image

before

image

after (this PR)

image

@rasvanjaya21
Copy link
Contributor Author

@dartpain Please review and merge. Thanks.

@dartpain dartpain merged commit d413d58 into arc53:main Oct 23, 2023
5 checks passed
@rasvanjaya21
Copy link
Contributor Author

Thank you @dartpain for accepting my PR, I'm happy to hear that you'd like to add the hacktoberfest-accepted label and holopin badge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants