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

[@mantine/nprogress] Init #1222

Merged
merged 9 commits into from
Apr 16, 2022
Merged

[@mantine/nprogress] Init #1222

merged 9 commits into from
Apr 16, 2022

Conversation

jerebtw
Copy link
Contributor

@jerebtw jerebtw commented Apr 10, 2022

This is a new version of my old PR (Complete rewrite)

  • Better prop names
  • Demos
  • Docs

New PR of old PR but based on the master branch

@jerebtw jerebtw mentioned this pull request Apr 10, 2022
4 tasks
@jerebtw
Copy link
Contributor Author

jerebtw commented Apr 10, 2022

@rtivital can you please review this PR (I'm not really good at writing documents)

@jerebtw
Copy link
Contributor Author

jerebtw commented Apr 13, 2022

Currently there is still one problem im not sure how to solve and that is when you want to restart the nprogress directly after it faded out because then it will not be visible for x ms (I will look in to that problem later this day)

eltongonc and others added 6 commits April 14, 2022 21:34
* [@mantine/hooks] use-fullscreen: custom ref state

Added a check to ensure a custom ref element toggles the fullscreen state

* fixed the prettier issue
…#1219)

* [@mantine/core] Select: only use open/close callback when value changes

* [@mantine/core] Select: do not use early return

* [@mantine/core] Tooltip: floating tooltip positioned inside component instead of popper

* [@mantine/core] Tooltip: revert positions story back

* [@mantine/core] Tooltip: remove popper from floating tooltip

* [@mantine/core] Tooltip: fix floating tooltip test, add position deps to coordinates array

* [@mantine/core] Tooltip: remove extra div
@jerebtw jerebtw changed the base branch from master to v5 April 14, 2022 22:08
@jerebtw
Copy link
Contributor Author

jerebtw commented Apr 15, 2022

I rebased the PR and updated the prop names

Currently there is still one problem im not sure how to solve and that is when you want to restart the nprogress directly after it faded out because then it will not be visible for x ms (I will look in to that problem later this day)

This is now also fixed

@rtivital Can you check the PR

@jerebtw jerebtw requested a review from rtivital April 15, 2022 16:05
@jerebtw jerebtw requested a review from rtivital April 15, 2022 18:25
@rtivital rtivital merged commit 0b4869e into mantinedev:v5 Apr 16, 2022
@rtivital
Copy link
Member

Thanks, great job!

@jerebtw jerebtw deleted the nprogress-init branch April 16, 2022 14:49
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.

4 participants