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

Bidi Support design proposal for (Numeric Shaping , National Calendar and UI Mirroring) #2178

Open
samarsultan opened this issue Feb 12, 2017 · 3 comments

Comments

@samarsultan
Copy link
Contributor

This is the proposed low level design of the three requested bidi features ( #2156 , #2154 and #1852 ).

  • Creation of bidi / js folders under notebook/static to wrap the whole bidi work as the below screenshot.

herierichy of bidi folder

  • Under bidi /js there will be two files:

    • bidi.js: the interface that hold the bidi function to be invoked.
    • numericshaping.js: has the core functions of numeric shaping support.

class diagram

  • Following is bidi.js in more details:
    The bidi API contains the following functions with respect to each feature:

bidi js functions for each feature

At the moment , I need any UI component to control the user preference if we enable the bidirectional behavior.

Thanks and Waiting your Comments .

@Carreau
Copy link
Member

Carreau commented Feb 27, 2017

Sorry I'm not subscribe to all the notification and missed this. I'll try to have a look, but I have plenty to do in the next 2 weeks.

@Carreau Carreau self-assigned this Feb 27, 2017
@Carreau
Copy link
Member

Carreau commented Feb 27, 2017

At the moment , I need any UI component to control the user preference if we enable the bidirectional behavior.

With master that's relatively easy to do, there is a command (Ctrl-Shift-P, toggle rtl layout) that allow you to set/unset the rtl flag on body. I've bound it on my system to Ctrl-r,t,l

screen shot 2017-02-27 at 10 14 16

I'll trust you on function naming and usage.

@jasongrout @blink1073 any feedback on the naming as we'd likely want to mirror this in TS/JupyterLab ?

@blink1073
Copy link
Contributor

The naming conventions look good to me. In TS we'd type the options as 'default' | 'national' | 'contextual', for example, but otherwise it should be the same.

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

No branches or pull requests

3 participants