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

RTL support #88

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

RTL support #88

wants to merge 3 commits into from

Conversation

Adidi
Copy link

@Adidi Adidi commented Sep 29, 2016

Explanation
if the page(body) or the scrolled component is in right to left (direction:rtl style) the default scrollbars are opposite and all the current calculation are wrong.
#69

Changes

  • add support for RTL component (src code)
  • add example in the examples page

To Do

  • there is a minor bug with firefox with the horizontal scrolling if its in RTL mode - the scrolling is not working smood (maybe i need help there)
  • add tests maybe for RTL

Before
before
After
after

All tests runs successfully after change

@abettadapur
Copy link

@malte-wessel Will this be reviewed/merged?

@Adidi
Copy link
Author

Adidi commented Nov 7, 2016

Anything ?

@joscha joscha mentioned this pull request Jun 2, 2017
@riseremi
Copy link

@malte-wessel any progress on this? Will be happy to have RTL support.

@atlanteh
Copy link

atlanteh commented Mar 1, 2018

@malte-wessel Can this be addressed?

@Adidi
Copy link
Author

Adidi commented Mar 1, 2018

@malte-wessel and this project has no value - since he is not cooperate at all !
Does anyone knows good alternative supports RTL ?

Thanks

@mouhsinelonly
Copy link

@malte-wessel will you take a look at this , we need rtl support

@pq1949
Copy link

pq1949 commented Apr 17, 2020

@atlanteh
Copy link

Can you please update the repo's homepage to your fork?

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.

None yet

6 participants