Skip to content
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

[Bug]: Rollover Overspending arrow icon obscured by scroll bar #2757

Closed
2 tasks done
matt-fidd opened this issue May 15, 2024 · 1 comment · Fixed by #2774
Closed
2 tasks done

[Bug]: Rollover Overspending arrow icon obscured by scroll bar #2757

matt-fidd opened this issue May 15, 2024 · 1 comment · Fixed by #2774
Labels
bug Something isn't working good first issue Good for newcomers regression This bug appeared after a recent release and needs to be fixed before the next release user interface Related to the user interface
Milestone
v24.6.0

Comments

@matt-fidd
Copy link
Contributor

matt-fidd commented May 15, 2024 •
edited

Verified issue does not already exist?

  • I have searched and found no existing issue
  • I will be providing steps how to reproduce the bug (in most cases this will also mean uploading a demo budget file)

What happened?

When rollover overspending is enabled, the indicator arrow is obscured by the scrollbar on the right.

Can be reproduced on edge.actualbudget.org with a test file, then enable rollover overspending on a category

image

What browsers are you seeing the problem on?

Chrome

Operating System

Windows 11
@matt-fidd matt-fidd added the bug Something isn't working label May 15, 2024
@youngcw
Copy link
Contributor

youngcw commented May 15, 2024

Looks like the arrow is shifted to the right compared what it used to be. That amount of space between the number and the scroll bar looks basically the same as normal

@MatissJanis MatissJanis added good first issue Good for newcomers help wanted Extra attention is needed user interface Related to the user interface regression This bug appeared after a recent release and needs to be fixed before the next release labels May 16, 2024
@MatissJanis MatissJanis added this to the v24.6.0 milestone May 17, 2024
Tigatok added a commit to Tigatok/actual that referenced this issue May 17, 2024
@Tigatok
fix: move rollover arrow into view
aebce30 
ref: actualbudget#2757
@Tigatok Tigatok mentioned this issue May 17, 2024
Merged
Tigatok added a commit to Tigatok/actual that referenced this issue May 17, 2024
@Tigatok
fix: move rollover arrow into view
9f7bd93 
ref: actualbudget#2757
@youngcw youngcw linked a pull request May 20, 2024 that will close this issue
fix: move rollover arrow into view #2774
Merged
joel-jeremy pushed a commit that referenced this issue May 21, 2024
@Tigatok
fix: move rollover arrow into view (#2774)
bbc123c 
* fix: move rollover arrow into view

ref: #2757

* fix: adds isNarrowWidth for responsive arrow

* ci: fix lint

* fix: update category name
@github-actions github-actions bot removed the help wanted Extra attention is needed label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers regression This bug appeared after a recent release and needs to be fixed before the next release user interface Related to the user interface
Projects
None yet
Milestone
v24.6.0
Development

Successfully merging a pull request may close this issue.

fix: move rollover arrow into view Tigatok/actual
3 participants
@MatissJanis @youngcw @matt-fidd