Double scrollbars in editor #1

Closed
opened 2021-05-07 20:37:46 -04:00 by merenber · 4 comments
merenber commented 2021-05-07 20:37:46 -04:00 (Migrated from git.uwaterloo.ca)

The editor has a scrollbar for the entire page and for the list of links on the left hand side. This might cause the user to not see links further down the list because part of the left scrollbar is hidden.
image

The editor has a scrollbar for the entire page and for the list of links on the left hand side. This might cause the user to not see links further down the list because part of the left scrollbar is hidden. ![image](/uploads/d65d15429638568a9562be62f3c6d17d/image.png)
merenber commented 2021-05-07 20:38:33 -04:00 (Migrated from git.uwaterloo.ca)

One possible solution is to make the editor take up 100% of the height of the screen.
Thoughts?

One possible solution is to make the editor take up 100% of the height of the screen. Thoughts?
a3thakra commented 2021-05-07 20:54:30 -04:00 (Migrated from git.uwaterloo.ca)

We could make the linklist preview on the right sticky (CSS position sticky) and the editor on the left should not have a height constraint. This way the editor would naturally increase the height of its parent and we would get a single scrollbar on the very right of the page (where it usually is)

We could make the linklist preview on the right sticky (CSS position sticky) and the editor on the left should not have a height constraint. This way the editor would naturally increase the height of its parent and we would get a single scrollbar on the very right of the page (where it usually is)
merenber commented 2021-05-07 22:08:37 -04:00 (Migrated from git.uwaterloo.ca)

Why not just remove 'height: 100%' from the outer div?

Why not just remove 'height: 100%' from the outer div?
a3thakra commented 2021-05-07 23:53:19 -04:00 (Migrated from git.uwaterloo.ca)

I haven't looked at it yet, and I'm not sure what div you're referring to. In any case, what you're saying sounds correct. :) I won't be able to get to it before Sunday, so I'll look at it then.

I haven't looked at it yet, and I'm not sure what div you're referring to. In any case, what you're saying sounds correct. :) I won't be able to get to it before Sunday, so I'll look at it then.
Sign in to join this conversation.
No Label
deployment
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: old/LinkList#1
No description provided.