-
Notifications
You must be signed in to change notification settings - Fork 354
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
BackToTop - full screen demo broken #10585
Comments
I think the double scrollbars is maybe from patternfly-react/packages/react-core/src/demos/examples/BackToTop/BackToTopNameDemo.tsx Lines 25 to 27 in bf19a67
|
Ah - I was seeking a demo to show me how to make it appear on scroll because that was behavior that seemed broken on org. so if that's not the intention here, i still have to figure out why it's not working on the org side... |
Discussed in a working session
|
The full screen demo is not displaying the backToTop component.
https://staging-v6.patternfly.org/components/back-to-top/react-demos/using-name-selector/
I've noticed that the component is also not being displayed on staging-v6.surge.sh, i'm not sure if it's due to the same issue that's broken this demo. A followup issue in org might be necessary.
it also has two different scrollbars, is that intended?
@srambach @andrew-ronaldson @mcoker
The text was updated successfully, but these errors were encountered: