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.

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

[FIX] Scroll Bar / Panel Width Adjustment cursor #2063

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

OwainWilliams
Copy link
Contributor

You can't scroll in the side panel, only make the panel wider.

Example of the issue.
scrollBarNotWorking

You can only make the panel wider.

  • Bug fix (non-breaking change which fixes an issue)

Motivation and context

From a users point of view, you can't scroll down to select any other items that might be visible in the left hand side.
The issue is the div with the class id divider-touch-area is sitting over the top of the scroll bar. The fix was to move this div 5px to the left and this then allows the user to scroll and still make the panel wider.

How to test?

See screenshot above, and the fix demo :

scrollBarWorking

Checklist

  • If my change requires a change to the documentation, I have updated the documentation in this pull request.
  • I have read the CONTRIBUTING document.

Copy link
Collaborator

@iOvergaard iOvergaard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@iOvergaard iOvergaard merged commit aedc0a8 into umbraco:main Jul 2, 2024
7 checks passed
@OwainWilliams
Copy link
Contributor Author

@iOvergaard Will this be in the next RC?

@iOvergaard
Copy link
Collaborator

@OwainWilliams There probably won't be any more RCs of 14.1 but it should land up in the final version of 14.1 👍

@iOvergaard
Copy link
Collaborator

Hi @OwainWilliams, I noticed this pull request was not included in the automated GitHub release notes. I have put it in manually, though, as the fix should be there in 14.1 released yesterday.

@OwainWilliams
Copy link
Contributor Author

@iOvergaard thanks. Ye, I tested 14.1 and it is there. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants