Open
Bug 1494587
Opened 7 years ago
Updated 3 years ago
Dynamically added <link rel="stylesheet alternate"> is not shown in the View > Page Style menu.
Categories
(Firefox :: Menus, defect, P5)
Firefox
Menus
Tracking
()
NEW
People
(Reporter: v.takeuchi, Unassigned)
References
()
Details
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:62.0) Gecko/20100101 Firefox/62.0
Build ID: 20180913170256
Steps to reproduce:
Add <link rel="stylesheet alternate"> element to the document:
<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<title>Sample</title>
</head>
<body>
<p>Lorem ipsum dolor sit amet.</p>
<button onclick="addStyleLink()">Add style</button>
<script>
function addStyleLink(){
const link = document.createElement("LINK");
link.title = "Insane";
link.rel = "stylesheet alternate";
link.href = "https://developer.mozilla.org/samples/cssref/altstyles/insane.css";
document.head.appendChild(link);
}
</script>
</body>
</html>
Actual results:
New style is not added to View>Page Style in Menu Bar immediately. After I click "No Style", the new style appears.
Expected results:
The new style appears immediately in the menu.
|
||
Comment 1•7 years ago
|
||
Thanks for filing! Is this a recent regression?
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(v.takeuchi)
|
||
Updated•7 years ago
|
Component: General → Menus
|
|
||
Updated•7 years ago
|
Summary: Dynamically added <link rel="stylesheet alternate"> is not shown in View>Page Style. → Dynamically added <link rel="stylesheet alternate"> is not shown in the View > Page Style menu.
|
||
Comment 2•7 years ago
|
||
I could be wrong, but I expect this has been broken at least since this was made to work at all in e10s mode.
There are perf concerns with the current implementation (see bug 1390928). I don't know if there's a reasonable solution here that doesn't make perf worse / harder to deal with, and also addresses this.
Priority: -- → P5
See Also: → 1390928
|
||
Comment 3•7 years ago
|
||
The solution of just sending a message to the child when the menu is open should address both concerns right?
|
|
||
Comment 4•7 years ago
|
||
(In reply to Emilio Cobos Álvarez (:emilio) from comment #3)
> The solution of just sending a message to the child when the menu is open
> should address both concerns right?
Yes, but doesn't work on macOS...
|
|
||
Updated•7 years ago
|
Flags: needinfo?(v.takeuchi)
|
||
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•