Bug 1894414 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Bug 1859349's refactoring of panelUI-shared.css turned this rule:
```css
panelmultiview[transitioning] > .panel-viewcontainer > .panel-viewstack > panelview > .panel-subview-body {
  overflow-y: hidden;
}
```
Into:

```css
panelview {
  &[transitioning] > .panel-viewcontainer > .panel-viewstack > panelview > .panel-subview-body {
    overflow-y: hidden;
  }
}
 ```
Which is subtly incorrect. The `transitioning` attribute is never added to `panelview`, only `panelmultiview`, so the selector never matches anything.

To reduce confusion, this broken rule should be removed.
Revision 1 by
on 
Bug 1859349's refactoring of panelUI-shared.css turned this rule:
```css
panelmultiview[transitioning] > .panel-viewcontainer > .panel-viewstack > panelview > .panel-subview-body {
  overflow-y: hidden;
}
```
Into:

```css
panelview {
  &[transitioning] > .panel-viewcontainer > .panel-viewstack > panelview > .panel-subview-body {
    overflow-y: hidden;
  }
}
 ```
Which is subtly incorrect. The `transitioning` attribute is only added to `panelmultiview`, not `panelview`, so the selector never matches anything.

To reduce confusion, this broken rule should be removed.

Back to Bug 1894414 Comment 0