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

Refactor RTL #141

Merged
merged 12 commits into from
Dec 3, 2020
Merged

Refactor RTL #141

merged 12 commits into from
Dec 3, 2020

Conversation

ronnetzer
Copy link
Contributor

@ronnetzer ronnetzer commented Nov 29, 2020

2 main things were done in this PR:

  1. expose dir from grid and use it internally for drag, resize and overlay plugins instead of material's Directionality
  2. pass the correct Directionality to markdown dynamic component portal from the current node injector (Dir directive instead of Directionality from root)

fixes #129
fixes #130

ronnetzer and others added 12 commits October 30, 2020 23:46
fix .pbl-ngrid-row-visually-hidden min-height being overridden when using pbl-ngrid-spacing mixin
fix pbl-ngrid-theme cell rtl border and positions
fix pbl-ngrid-spacing rtl paddings
fix drag rtl position and style (resizer handler)
fix header context menu rtl position
expose grid directionality
use grid's dir in column resize and meta-rows' horizontal scroll calculations
add dir to DynamicColumnWidthLogic
use dir from grid in drag plugin before drag starts
use dir from grid in overlay-panel
fix meta rows horizontal scroll in rtl
shlomiassaf added a commit that referenced this pull request Dec 3, 2020
This commit is a hard copy of [#141](https://github.com/shlomiassaf/ngrid/pull/141/files) which was done on master (v2) and is almost imposible to merge into v3 branch due to the amount of changes

Thanks @ronnetzer
@shlomiassaf shlomiassaf changed the base branch from master to 2.x.x December 3, 2020 20:05
@shlomiassaf shlomiassaf merged commit 1520f29 into shlomiassaf:2.x.x Dec 3, 2020
shlomiassaf added a commit that referenced this pull request Dec 3, 2020
This commit is a hard copy of [#141](https://github.com/shlomiassaf/ngrid/pull/141/files) which was done on master (v2) and is almost imposible to merge into v3 branch due to the amount of changes

Thanks @ronnetzer
shlomiassaf added a commit that referenced this pull request Dec 3, 2020
This commit is a hard copy of [#141](https://github.com/shlomiassaf/ngrid/pull/141/files) which was done on master (v2) and is almost imposible to merge into v3 branch due to the amount of changes

Thanks @ronnetzer
shlomiassaf added a commit that referenced this pull request Dec 3, 2020
This commit is a hard copy of [#141](https://github.com/shlomiassaf/ngrid/pull/141/files) which was done on master (v2) and is almost imposible to merge into v3 branch due to the amount of changes

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

Successfully merging this pull request may close these issues.

group width calculation in RTL mode is invalid drag plugin does not factor RTL layout
2 participants