mirror of
https://github.com/actualbudget/actual.git
synced 2026-03-10 04:02:38 -05:00
[Bug]: Double scrollbars #512
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/actual#512
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @MatissJanis on GitHub (Jul 18, 2023).
Verified issue does not already exist?
What happened?
There are 2x scrollbars in many pages (rules, schedules, budget, transactions, etc.). We should only have 1x scrollbar.
What error did you receive?
No response
Where are you hosting Actual?
None
What browsers are you seeing the problem on?
No response
Operating System
None
@Avitus27 commented on GitHub (Jul 18, 2023):
Hi @MatissJanis I'd like to help with this issue. Could you add info on the browser and OS you're seeing this in so I can reliably replicate this? TIA!
@Kidglove57 commented on GitHub (Jul 18, 2023):
Just to say that I am not personally seeing this on 23.7. 2. I have tried in Firefox and Safari. Nor am I seeing it in Netlify Preview.
@MatissJanis commented on GitHub (Jul 18, 2023):
Latest chrome + OSX + always show scrollbars
@Kidglove57 commented on GitHub (Jul 18, 2023):
Thanks for confirming. I am on the latest OSX with "Always" enabled in OSX system settings for scroll bars.
I confirm that I am not seeing a double scroll bar in Chrome. Happy to help though with any info to troubleshoot this.
Here is what I see:
@MatissJanis commented on GitHub (Jul 18, 2023):
Do you have the browser maximized? It seems to be working correctly with the window mazimized, but if it's not maximized (and am using a laptop screen), then the two scrollbars appear.
@Kidglove57 commented on GitHub (Jul 18, 2023):
Good call. I can reproduce this now BUT:
I was struggling for the right terminology there so hope it makes sense. In other words when the browser is in landscape view rather than portrait and reduced in height by about half.
@Kidglove57 commented on GitHub (Jul 18, 2023):
In that specific sizing and orientation I am also reproducing this on Firefox and Safari.
Hope that helps.
@Avitus27 commented on GitHub (Jul 19, 2023):
I was able to reproduce under these settings & environment:
I wasn't seeing this at first because I had my zoom set to 80%
I've also narrowed it down to a CSS style
.css-f0f5zh{ min-height:700px;}, unchecking this in inspector removes the double scrollbar. Had a look around the code but I've no clue about Glamor, which seems to be how that CSS is generated. Sorry I can't be of more help here! 😓@aleetsaiya commented on GitHub (Jul 22, 2023):
I thought the purpose of the outer scrollbar is to make user to be able to scroll down to the page (not transaction table). When remove the
min-height: 700px, it can make the outer scrollbar disappeared, but user will not be able to scroll down the whole page.Do you think should we remove the outer scrollbar? This will make small screens user not able to scroll down to hide the title section (like "All Accounts" in the demo video)
Demo when I remove the
min-heightlimit:https://github.com/actualbudget/actual/assets/67775387/912ec105-274e-480b-bde5-541004ac9234
@MatissJanis commented on GitHub (Jul 22, 2023):
Is there a way to set this according to the screen height? i.e. for smaller screens we would have the 2x scrollbars, but for bigger screens (think: all laptops and bigger) - we remove the double scrollbar.