<div dir="ltr"><div>Thanks Itiel,</div><div><br></div><div>9. Yes, you're right. I looked at it wrong when writing this up.</div><div>10. Well, if we think about moving something "forward", then I would assume that it should follow the direction of locale text.</div><div>14. Yes you're right, the axis are showing the data in the correct quadrant.</div><div><br></div><div>Cheers,</div><div>Jared<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 8, 2020 at 1:19 PM Itiel Yossefian <<a href="mailto:itiel_yn8@walla.co.il">itiel_yn8@walla.co.il</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div dir="rtl"><div dir="ltr">
<div dir="ltr">Jared, thanks for testing!</div>
<div dir="ltr"> </div>
<div dir="ltr">It seems that `intl.l10n.pseudo` is a bit buggy atm (at least, for the sake of testing RTL), and most of your mentions are a non-issue on a real RTL build or when testing with `intl.uidirection`, so I'll address your valid mentions only.</div>
<div dir="ltr">I'll file bugs for fixing intl.l10n.pseudo.</div>
<div dir="ltr"> </div>
<div dir="ltr">5. The icon does face the "right way" when testing with `intl.uidirection`, but either way I don't think this matters for RTL. I mean, the direction in which data comes wirelessly into a device can be from either direction...</div>
<div dir="ltr">6. That's <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1529091" target="_blank">bug 1529091</a>. The MDN page meant to mirror the magnifying glass icon only when it is <b>originally</b> rendered as if it being held by someone left-handed (see it's linked icon). Across the Firefox LTR UI, there is inconsistency with this icon. Some render it as left handed and others as right handed. I'm not sure how most of world actually hold a magnifying glass, but I think this should be unified across the UI.</div>
<div dir="ltr">But that's a topic for different discussion, I guess :)</div>
<div dir="ltr">7. I think you're looking for <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1539633" target="_blank">bug 1539633</a>.</div>
<div dir="ltr">8. Huh, never thought of that, thanks. Will file a bug about that. Now that you mention it, the same can be said about the Reader View icon...</div>
<div dir="ltr">9. Can you elaborate? Tested also with `intl.l10n.pseudo` and it seems to look fine. Was fixed in <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1279230" target="_blank">bug 1279230</a> and <a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1594833" target="_blank">bug 1594833</a>.</div>
<div dir="ltr">10. See #5 for the "Send tab to device"; "Pin" looks fine when testing with `intl.uidirection`. I'm not sure about the "Share" icon... It's not like you're sharing the tab to the left or to the right :)</div>
<div dir="ltr">14. Good question about the protections report icon. I think it's fine as-is because the way it looks now, all the numbers can be in the positive section, while if you'll mirror the icon, they'll be in the negative-positive section. I don't think you can have negative numbers of trackers :)</div>
<div dir="ltr"> </div>
<div dir="ltr">Thanks again!</div>
<div dir="ltr">Itiel</div>
</div></div><blockquote style="margin:0px 0px 20px;border-top:1px solid rgb(224,224,224)"><br>ב פבר׳ 7, 2020 18:36, Jared Wein כתב:<blockquote style="margin:0px 0px 20px;border-top:1px solid rgb(224,224,224)">
<div dir="ltr"><div>I have taken a quick look through the Firefox UI and believe the following issues are present with RTL support (tested with 
intl.l10n.pseudo=bidi in about:config, then restarting):</div><div><br></div><div>1. The "Show Sidebars" icon in the navigation toolbar should be flipped in RTL.</div><div>2. The arrows for submenus in the menupopup for the Library button in the navigation toolbar are facing the wrong way.</div><div>3. The arrows for submenus in the Firefox menu are facing the wrong way.</div><div>4. The "external link" icon for Firefox Monitor and Firefox Send, found within the Account submenu of the Firefox menu are facing the wrong way.</div><div>5. The "send tab to device" icon in the Account submenu of the Firefox menu is facing the wrong way.</div><div>6. The magnifying glass in about:config is flipped when we normally wouldn't flip an icon that represents something from the physical world that is held by the majority of people in their right hand. In the linked MDN page has this topic in the "what to mirror" section, though the sub-referenced "material design" page mentions this as something to "not to mirror".<br></div><div>7. The background for the tracker doorhanger's expanded info section in the location bar should have the shield on the left side in RTL.</div><div>8. The "Print" icon should be flipped since the paper coming out of the printer shows directional text.</div><div>9. The Pocket menu does not appear to have any RTL support.</div><div>10. The "Pin tab", "Send tab to device", and "share" icons in the "..." menu of the location bar are facing the wrong way.</div><div>11. The "Go" arrow in the search box that appears after the user starts typing is facing the wrong way in RTL.</div><div></div><div>12. The "Last Page", "Next Page", "Previous Page", "First Page" icons in Print Preview are facing the wrong way in RTL.</div><div>13. The submenu arrows in the Site Information popup are facing the wrong way.<br></div><div><br></div><div>Icons I'm not sure about:</div><div>1. The "protections report" icon shows a graph with the Y axis on the left side. Should this be flipped for RTL?</div><div><br></div><div>Many of these would make good-first-bugs for people to mentor. Can someone look through this list and let me know if I got any of them wrong?</div><div><br></div><div>Thanks,</div><div>Jared<br></div><div><br></div></div><br><div><div dir="ltr">On Thu, Feb 6, 2020 at 7:04 PM Tim Nguyen <<a href="mailto:ntim.bugs@gmail.com" target="_blank">ntim.bugs@gmail.com</a>> wrote:<br></div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hello folks,</div><div><br></div><div>There has been a significant number of RTL bugs found in the last few months in the Firefox UI. From what I can gather, they were mostly due lack of awareness on specific situations that are more complex than just using CSS logical properties, but sometimes due to lack of testing.<br></div><div><br></div><div>Itiel has made a great job at filing+fixing many bugs and wrote up a guide based on his experience. This guide describes in detail what things should be mirrored, text handling in RTL and tips on how to spot/fix/test RTL issues. It includes things I personally didn't know about before, so I do encourage giving it a read:</div><div><br></div><div><a href="https://developer.mozilla.org/docs/Mozilla/Developer_guide/RTL_guidelines" target="_blank">https://developer.mozilla.org/docs/Mozilla/Developer_guide/RTL_guidelines</a></div><div><br></div><div>When developing a new part of the UI or making significant UI changes, please do make sure you manually test RTL for issues (by setting
 <span dir="auto"><code><span>intl.l10n.pseudo<span style="font-family:arial,sans-serif"> to </span>bidi<span style="font-family:arial,sans-serif">) to make Firefox accessible for all languages :)<br></span></span></code></span></div><div><br><span dir="auto"><code><span><span style="font-family:arial,sans-serif"></span></span></code></span></div><div>I hope that this guide will be helpful, and please do let me or Itiel know if you've got any feedback on it.<br><span dir="auto"><code><span><span style="font-family:arial,sans-serif"></span></span></code></span><span dir="auto"><code><span></span></code></span></div><div><div><div><br></div><div>Cheers,<br></div><div>Tim,</div><div>Firefox Desktop Theme peer<br></div><div><br></div></div></div></div>
_______________________________________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org" target="_blank">firefox-dev@mozilla.org</a><br>

<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listinfo/firefox-dev</a><br>
</blockquote></div>

<br></blockquote></blockquote><br></div>
</blockquote></div></div>