angular-ui/ui-grid

RTL horizontal scrolling is broken while moving columns

Open

#5,177 opened on Mar 1, 2016

View on GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (5,395 stars) (2,496 forks)batch import
help wanted

Description

Scrolling the grid while moving columns doesn't seem to be working for RTL. I'm running Windows 10 and have seen the issue in Chrome, FF, and IE11. Trying to move a column to the left doesn't scroll at all, but then when I drag the same column back towards the right side of the grid, the grid scrolls to the left. Here's a plunkr (based on the example in the column moving tutorial) demonstrating this.

  1. Click and hold the right-most column header ("Id" in this case).
  2. Drag the column to the left side of the grid.

Notice that the grid did not scroll to the left even though it should have. Now, if you drag the column back to the right side of the grid, you can see the grid scroll to the left when it shouldn't have scrolled at all.

Contributor guide

Tech stack
javascriptangular
Domain
frontendlocalization
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
familiarity with RTLunderstanding of ui grid column moving
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65
Research direction
Investigate the column moving drag behavior in RTL mode using the provided plunkr. The issue indicates that dragging a column to the left does not scroll, while dragging to the right scrolls left. Check the scroll logic in the column moving source files (e.g., ui grid column moving.js) and ensure that the scroll direction is reversed for RTL. Look at two comments for any additional insights or code suggestions.