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

added getCurrentUrl function & url in onClose Callback #24

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

Yusuf-Uluc
Copy link

Needed these changes for my project so feel free to merge it to the main repo in case you want to :)

@BullsEye34
Copy link

@Yusuf-Uluc I tried your git. The changes are nice, really cool.
I noticed that the changes are only for Sheet. Can it be implemented for Modal as well?

And can the URL Text color be changed? For some reason my URL color was completely white.

@Yusuf-Uluc
Copy link
Author

Hey @BullsEye34 glad you like the changes. Right now I sadly don't have a lot of time so I don't think I'll be implementing anything more for now. But I am sure that it is possible. When it comes to the URL you can change the text color, currently it is set to white.

@BullsEye34
Copy link

Ahh I see @Yusuf-Uluc

That's cool. I really appreciate the work though. Kudos!

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.

2 participants