Addons-server: Color selectors in static theme submission wizard have layout issues in RTL

Created on 13 Mar 2018  Â·  22Comments  Â·  Source: mozilla/addons-server

STR:

  1. Log into Developer Hub and submit a new static theme using the wizard
  2. Switch to an RTL locale
  3. Observe the text layout for the color selectors

Actual result:
The color selector area does not have the same margins as the LTR locales

Expected result:
Color selector area has the same margins for RTL and LTR

Notes:

  • reproduced on AMO-dev, FF58 Win10x64

image

devhub i18n assigned good first bug mentor assigned p4 verified fixed

All 22 comments

Mentor: @eviljeff

If this is your first contribution, please refer to https://wiki.mozilla.org/Add-ons/Contribute/Code on how to get started.

I want to work on this issue.

@deepanshu-jain1999 great - this should just be a css fix

Thanks

On Tue, 24 Apr 2018, 4:03 pm Andrew Williamson, notifications@github.com
wrote:

@deepanshu-jain1999 https://github.com/deepanshu-jain1999 great - this
should just be a css fix

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/addons-server/issues/7783#issuecomment-383884411,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Adum0amySeGwuyxhs702ldGROsAFmaJOks5trv92gaJpZM4Sogbp
.

It seems in this way on linux .

issue

@deepanshu-jain1999 that's the old lightweight theme wizard - the one this issue is about is the new static theme wizard https://addons-dev.allizom.org/en-US/developers/addon/submit/wizard-listed

Thanks I got it . Actually I am new to open source. So I feel afraid how to do it. Please help me where I will get this issue in project. Is this problem specific to window 10 user because it seems fine on linux

newissue

@deepanshu-jain1999
The problem isn't OS specific, but is specific to RTL locales (e.g. Arabic, Hebrew, Urdu, etc). I should have linked to one of those pages to demonstrate, e.g. https://addons-dev.allizom.org/ar/developers/addon/submit/wizard-listed

Thanks I got it. It should be in this style
solvedwithspace

I have done it on browser . Now I am going to implement it inside project

When I signup on the project locally it redirect to "refuse to connect ". What am i doing wrong?

That's quite a vague error and impossible to debug without more details. If you visit the #amo irc channel we might be able to suggest something.

Yes I have posted on http://disq.us/p/1s9bdf6

Yes I have posted on http://disq.us/p/1s9bdf6

That's not irc... that's a website with a directory listing of irc channels.

I have solved login problem but I am unable to access page http://olympia.test/en-GB/developers/addon/submit/wizard-listed locally. It does not show "Create a Theme Version" div on http://olympia.test/en-GB/developers/addon/submit/upload-listed

ah, yes, it's turned off by default. You can enable it by adding a flag (a "waffle switch") in django admin here: http://olympia.test/admin/models/waffle/switch/ called "allow-static-theme-uploads". It will look something like this:
screenshot 2018-05-14 19 08 20

Thanks

Hey there @deepanshu-jain1999, how's it going with this issue?

I am in doubt that how to switch RTL locale

I linked the page in RTL local in https://github.com/mozilla/addons-server/issues/7783#issuecomment-386048998

I create a pull request for this issue. Please review it.

8463 Fix issue #7783

Verified as fixed on AMO-dev with FF60, WIn10x64

image

Was this page helpful?
0 / 5 - 0 ratings