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

Allowing keyboard input for the ScrollViewer #471

Merged
merged 1 commit into from
Jul 20, 2020

Conversation

RBrid
Copy link
Contributor

@RBrid RBrid commented Jun 11, 2020

User could not use Ctrl +/- to zoom in and out of the ScrollViewer in the ScrollViewer sample page.
Arrows, PageUp, PageDown, etc did not work either to scroll its Image. The IsTabStop=True setting allows the control to take focus & keyboard input works now.

This is for internal bug 26337703 - Cannot set focus to ScrollViewer.

Testing

Ad-hoc testing of the ScrollViewer sample page.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

@RBrid RBrid added the bug Something isn't working label Jun 11, 2020
@RBrid RBrid self-assigned this Jun 11, 2020
@marcelwgn
Copy link
Collaborator

marcelwgn commented Jun 11, 2020

Is it expected that when HorizontalScrollBarVisibility is set Hidden, that scrolling to the left and right doesn't work persistently:

gif

Edit: In the gif above, scrolling was down through arrow keys. Scrolling up and down works, however scrolling left and right jumps to the previous location once the left/right arrow key is lifted.

@RBrid
Copy link
Contributor Author

RBrid commented Jun 11, 2020

@chingucoding, no that is not normal & I could not repro this. @stmoy, can you repro this? Thanks.

@RBrid RBrid requested a review from stmoy June 22, 2020 22:02
@RBrid
Copy link
Contributor Author

RBrid commented Jun 22, 2020

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@stmoy
Copy link
Contributor

stmoy commented Jul 18, 2020

@RBrid - I am able to reproduce this issue.

Steps:

  1. On the ScrollViewer page in the Xaml Controls Gallery app
  2. Set ZoomMode = Enabled
  3. Zoom in a bit using the Zoom slider
  4. Set ScrollbarVisibility.Horizontal = Visible, or Hidden
  5. Tab to the image to give it focus
  6. Hold the down arrow key to scroll the image down
  7. Release the down arrow key and quickly press the right arrow key

Expected:
Image starts scrolling to the right from the current vertical position

Actual:
Image starts scrolling to the right from the top of the image (ie. it looks like it jumps to the top)

Note that if you wait a second between pressing down and pressing right, the behavior does not repro for me.

@stmoy
Copy link
Contributor

stmoy commented Jul 20, 2020

Followed up with @RBrid - it looks like this is indeed a framework issue and should not block this PR.

@stmoy stmoy merged commit 34aa38e into master Jul 20, 2020
@bpulliam bpulliam deleted the user/regisb/ScrollViewerTabStopFix branch May 10, 2022 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants