Skip to content

Conversation

@GuillaumeGomez
Copy link
Member

@GuillaumeGomez GuillaumeGomez commented Aug 25, 2021

Fixes #1474

Peek.2021-08-25.14-59.mp4

Question now: do you want to put the "hide files" button outside of the scrolled area so it always remain on top or not?

@GuillaumeGomez GuillaumeGomez requested a review from jyn514 August 25, 2021 09:23
@GuillaumeGomez GuillaumeGomez force-pushed the source-file-sidebar-scroll branch from d650746 to afde9ea Compare August 25, 2021 09:31
@Nemo157
Copy link
Contributor

Nemo157 commented Aug 25, 2021

It doesn't look like this works to me? https://ipvold-docs-rs-dev.nemo157.com/crate/dialectic-tokio-serde/0.1.0/source/Cargo.lock still has the list stuck to the top of the page.

Also https://ipvold-docs-rs-dev.nemo157.com/crate/dialectic-tokio-serde/0.1.0/source/ is empty.

@GuillaumeGomez
Copy link
Member Author

I can't see your pages for "dialectic-tokio-serde". Also: the scrollbar only appears when the source file list is longer than the source code.

@GuillaumeGomez
Copy link
Member Author

Ah hold on! Just understood: the issue meant that we want to have the source files list always visible whatever the scroll position in the source file. Ok, I'll update.

@GuillaumeGomez GuillaumeGomez force-pushed the source-file-sidebar-scroll branch from afde9ea to 5ca05a1 Compare August 25, 2021 12:59
@GuillaumeGomez
Copy link
Member Author

Ok, updated. I discovered position: sticky. That was quite the ride. I added a video too.

@Nemo157
Copy link
Contributor

Nemo157 commented Aug 25, 2021

Is it possible to not have it scroll off the top when you scroll to the very bottom?

@GuillaumeGomez
Copy link
Member Author

Not that I know of. The footer is "pushing" the content up, which is why the sticky position is being disabled at that time.

@Nemo157
Copy link
Contributor

Nemo157 commented Aug 25, 2021

On https://ipvold-docs-rs-dev.nemo157.com/crate/tokio/1.3.0/source/src/io/util/async_buf_read_ext.rs I need to scroll the page all the way down to see the last few entries in the file list. I wonder if it should be reduced in height to account for the entire header block (which might also fix the footer pushing it up). That might leave a bit of blank space below it, but that seems better than not being able to access the files.

@GuillaumeGomez
Copy link
Member Author

Oh nice! That's a bug. :)

@GuillaumeGomez GuillaumeGomez force-pushed the source-file-sidebar-scroll branch from 5ca05a1 to 7a56f95 Compare August 25, 2021 13:43
@GuillaumeGomez GuillaumeGomez force-pushed the source-file-sidebar-scroll branch from 7a56f95 to 9d833dc Compare August 25, 2021 13:46
@GuillaumeGomez
Copy link
Member Author

Fixed the bug, great catch, thanks!

@GuillaumeGomez
Copy link
Member Author

Anything else to be fixed @Nemo157 ?

@jyn514
Copy link
Member

jyn514 commented Sep 5, 2021

This is great, thanks :)

@jyn514 jyn514 merged commit fac3ea5 into rust-lang:master Sep 5, 2021
@GuillaumeGomez GuillaumeGomez deleted the source-file-sidebar-scroll branch September 5, 2021 14:36
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.

"Hide files" toggle should be fixed so it isn't hidden after scrolling

3 participants