?
avatar Quy
Quy
6 Feb 2020

Steps to reproduce the issue

Use Microsoft Edge.
Install Persian language.
Edit account.
Switch to Persian language for the backend language.
Click Save & Close.
Edit account.
See missing toolbar.
This is happening with toolbars in other areas.

Expected result

toolbar-ltr

Actual result

toolbar-rtl

System information (as much as possible)

Microsoft EdgeHTML 18.18363

avatar Quy Quy - open - 6 Feb 2020
avatar joomla-cms-bot joomla-cms-bot - change - 6 Feb 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 6 Feb 2020
avatar brianteeman
brianteeman - comment - 6 Feb 2020

So this is the non-chromium edge?

avatar Quy
Quy - comment - 6 Feb 2020

How do I find out? This comes preinstalled with Windows 10 Home.

avatar brianteeman
brianteeman - comment - 6 Feb 2020

I cant tell you as I already upgraded to the chromium release https://www.microsoft.com/en-us/edge

avatar Quy
Quy - comment - 6 Feb 2020

Upgraded and now it is fine. Should non-chromium Edge be supported?

avatar brianteeman
brianteeman - comment - 6 Feb 2020

That I don't know. I would create a new issue regarding that and then @wilsonge can direct a decision

avatar wilsonge
wilsonge - comment - 8 Feb 2020

This is a decision for production to make as a whole - my personal opinion would be not to support it

// cc @HLeithner

avatar HLeithner
HLeithner - comment - 8 Feb 2020

Our current support cycle is planned Major version - 2, MS forces all users to upgrade to blink rendering version but I didn't found the timeline for the eol of EdgeHTML. So I would suggest to fix this problem instead hoping that EdgeHTML is not longer supported by MS.

avatar jwaisner jwaisner - change - 8 Feb 2020
Status New Confirmed
Build staging 4.0-dev
avatar Quy Quy - change - 9 Feb 2020
Title
[4.0] Missing toolbar with RTL in Edge
[4.0] Missing toolbar with RTL in EdgeHTML
avatar Quy Quy - edited - 9 Feb 2020
avatar Quy
Quy - comment - 11 Feb 2020

OK the issue is with position: sticky in RTL with EdgeHTML.
@ciar4n Can you suggest a fix? Thanks.

.subhead {
  position: sticky;
  top: 0;
  right: 0;
  left: 0;
  z-index: $zindex-toolbar;
  width: auto;
  min-height: 43px;
  padding: 13px 0;
  color: var(--atum-text-dark); //#0c192e;
avatar ciar4n
ciar4n - comment - 11 Feb 2020

This appears to be the case... https://stackoverflow.com/questions/56835658/in-microsoft-edge-sticky-positioning-doesnt-work-when-combined-with-dir-rtl

Until RTL/sticky is fixed in Edge, maybe something like...

@supports (-ms-ime-align:auto) {
  [dir=rtl] .subhead {
    position: relative;
  } 
}
avatar Quy
Quy - comment - 11 Feb 2020

Please test PR #27897

avatar Quy Quy - close - 11 Feb 2020
avatar Quy Quy - change - 11 Feb 2020
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2020-02-11 18:44:38
Closed_By Quy

Add a Comment

Login with GitHub to post a comment