Skip to content

Going back in the dartdocs loses your scroll position #10984

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.

Sign up for GitHub

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

Jump to bottom
Closed
Hixie opened this issue Jun 27, 2017 · 7 comments
Closed

Going back in the dartdocs loses your scroll position #10984

Hixie opened this issue Jun 27, 2017 · 7 comments
Labels
c: regression It was better in the past than it is now d: api docs Issues with https://api.flutter.dev/ will need additional triage This issue or PR needs attention before being routed to secondary triage

Comments

@Hixie
Copy link
Contributor

Hixie commented Jun 27, 2017

This used to be fine. I think it broken when we added scrolling in the left and right columns.

Steps:

  1. Go to https://master-docs-flutter-io.firebaseapp.com/flutter/rendering/ContainerRenderObjectMixin-class.html
  2. Open "add".
  3. Nope, that was the wrong one. Go back.
  4. Open "addAll".
  5. Nope, that was the wrong one. Go back.
  6. Get frustrated that you had to scroll after step 3 and have to scroll again now.

cc @sethladd
@Hixie Hixie added dev: docs - website c: regression It was better in the past than it is now labels Jun 27, 2017
@Hixie Hixie added this to the No milestone necessary milestone Jun 27, 2017
@Sfshaza Sfshaza added d: api docs Issues with https://api.flutter.dev/ and removed d: website - content labels Jul 3, 2018
@Sfshaza
Copy link

Sfshaza commented Jul 3, 2018

Still broken.

@zoechi
Copy link
Contributor

zoechi commented Dec 5, 2018

Is this d: website - infra or is this something dartdoc needs to fix?

@zoechi zoechi added the will need additional triage This issue or PR needs attention before being routed to secondary triage label Dec 17, 2018
@zoechi
Copy link
Contributor

zoechi commented Dec 17, 2018

@Hixie I don't know what to do with dartdoc infra issues.
Should they be moved to dart-lang/dartdoc?

@Hixie
Copy link
Contributor Author

Hixie commented Jan 16, 2019

@zoechi yeah probably.
cc @jcollins-g

@jcollins-g
Copy link
Contributor

@zoechi Yes, move to dart-lang/dartdoc and I will be more likely to see/address them.

@zoechi zoechi mentioned this issue Jan 30, 2019
Closed
@Hixie
Copy link
Contributor Author

Hixie commented Jan 30, 2019

Since we have moved the bug I'll close this one. Thanks all!

@Hixie Hixie closed this as completed Jan 30, 2019
@github-actions
Copy link

github-actions bot commented Aug 8, 2021

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
c: regression It was better in the past than it is now d: api docs Issues with https://api.flutter.dev/ will need additional triage This issue or PR needs attention before being routed to secondary triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@zoechi @Hixie @Sfshaza @jcollins-g