[bisq-network/bisq] add Right-to-Left support (#2774)
Christoph Atteneder
notifications at github.com
Thu Apr 25 12:37:05 UTC 2019
ripcurlx requested changes on this pull request.
NACK - based on my inline comments. These required changes could be easily implemented, but I recognized that there are some issues with the tabs and toggle buttons we are using.
<img width="1312" alt="Bildschirmfoto 2019-04-25 um 14 28 45" src="https://user-images.githubusercontent.com/170962/56736027-5c56de00-6767-11e9-87e3-b780983cb36f.png">
Tab labels are squashed and hardly readable and the toggle is slightly misplaced and squashed as well. I'll see if I update the library if the problems were fixed already within newer versions.
> @@ -84,6 +84,12 @@
*/
);
+ public static final List<String> rtlLanguagesCodes = Arrays.asList(
Can be private
> @@ -167,6 +169,8 @@ public MainView(MainViewModel model,
@Override
protected void initialize() {
MainView.rootContainer = root;
+ if (LanguageUtil.isDefaultLanguageRTL())
+ MainView.rootContainer.setNodeOrientation(NodeOrientation.RIGHT_TO_LEFT);
Please apply Bisq coding conventions and style guide: https://github.com/bisq-network/bisq/blob/master/CONTRIBUTING.md#style-and-coding-conventions
> @@ -165,6 +165,9 @@ bg color of non edit textFields: fafafa
-fx-selection-bar-non-focused: -fx-selection-bar;
-fx-font-size: 13;
+}
+
+.root:dir(ltr){
Why was this necessary?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/2774#pullrequestreview-230612058
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190425/407fc020/attachment.html>
More information about the bisq-github
mailing list