Open
Bug 36511
Opened 25 years ago
Updated 2 years ago
Support LinkStyle and dynamic stylesheet changes on <meta http-equiv="Link">
Categories
(Core :: DOM: CSS Object Model, defect, P5)
Core
DOM: CSS Object Model
Tracking
()
NEW
People
(Reporter: jst, Unassigned)
References
Details
(Keywords: dom2)
Attachments
(1 file, 2 obsolete files)
(deleted),
text/html
|
Details |
No description provided.
Reporter | ||
Comment 1•25 years ago
|
||
Done for XML PI's, HTMLLinkElement still to be done.
Status: NEW → ASSIGNED
Comment 2•25 years ago
|
||
Does this need to be nsbeta2 or can we drop for FCS?
Reporter | ||
Comment 3•24 years ago
|
||
This could be very useful, but IMO not absolutely critical for FCS, I think
we'll get this more or less for free once bug 7515is fixed. nominating for beta2
Keywords: nsbeta2
Updated•24 years ago
|
Component: DOM Level 2 → DOM Style
Comment 6•24 years ago
|
||
Taking QA Contact on all open or unverified DOM Style bugs...
QA Contact: vidur → ian
Comment 7•24 years ago
|
||
Nominating this bug for nsbeta1 on behalf of gerardok@netscape.com.
Keywords: nsbeta1
Comment 8•24 years ago
|
||
This seems to work for HTML Link elements and STYLE elements, but fails for META
elements. Since this means it mainly works, removing nsbeta1 nomination.
Keywords: nsbeta1
Comment 9•22 years ago
|
||
So what needs to be done here?
Comment 10•22 years ago
|
||
IE displays the h1 as red. I *believe* Mozilla should too...
I think this is related to bug #7515, but that's marked as VERIFIED/FIXED and,
to judge by this test case - it isn't :(
I really don't like seeing IE do something better than Mozilla :,-( Sorry I
don't know enough about Mozilla to help debug it.
J.
Comment 11•22 years ago
|
||
The reason my code didn't work is because the base uri of dynamically generated
html is not the same as the base uri of the generating page.
Should it be?
Comment 12•22 years ago
|
||
Good question. Radha? What does NS4 do?
Comment 13•22 years ago
|
||
Based on what jrgm has said in comment #11, when a wyciwyg url is loaded
normally, the generating document is not set as the base url. However this is
done if a wyciwyg url is loaded from history. I looked into making the
generating url as the base url for all types of loads, instead of just for
history loads. But there are some issues with it in docshell. I think it can be
worked out. But I haven't seen a compelling reason to do it.
Comment 14•22 years ago
|
||
What would you consider a "compelling reason"? I might be able to supply one...
Comment 15•22 years ago
|
||
I may consider a "loss of data" or a top 100 site rendering failure as a
compelling reason.
Comment 16•22 years ago
|
||
Shame, I've got no way to offer you something that serious.
I do have a work around. I'd rather not have too, but I can't add anymore
reasoning.
Sorry to have bothered you!
J.
Comment 17•22 years ago
|
||
Radha, could you at least outline the issues that docshell has with setting the
base URI to the generating URI? Just so those who feel that this being a bug is
compelling enough a reason to fix it can actually do so.
Comment 18•22 years ago
|
||
Just as a quick note, I think I've seen at least 2 or 3 bug reports caused by
this "no compelling reason to fix it" bug.. I agree with Christopher; some
details on the technical issues would be nice.
Comment 19•22 years ago
|
||
The only issue I'm aware of is that this fails for <meta> elements that link in
stylesheets.
Comment 20•22 years ago
|
||
Hixie, this seems to work just fine to me....
Comment 21•22 years ago
|
||
The issues in docshell relate to how pages are loaded and added to Session
History . Search for "OriginalURI" in docshell and look through the flow of
control before a page is sent to necko in DoChannelLoad() and after the
onStartRequest response from necko comes, from OnLoadingSite() to OnNewURI()
and eventually to SetCurrentURI(). When original url is set different from the
actual loaded url, nsDocShell::mCurrentURI will end up with a different value
than what it has actually loaded. This could lead to a slew of other problems
when subsequent urls are loaded and when user navigates through history to and
from this wyciwyg url.
Comment 22•22 years ago
|
||
bz: but does that <meta> element support LinkStyle?
Comment 23•22 years ago
|
||
Attachment #99505 -
Attachment is obsolete: true
Attachment #100411 -
Attachment is obsolete: true
Comment 24•22 years ago
|
||
May as well take this... Let's have a separate bug for the wyciwyg discussion,
ok, since there's work left to do here on the original problem...
Assignee: jst → bzbarsky
Status: ASSIGNED → NEW
Comment 25•22 years ago
|
||
This needs to be fixed in order to ensure correct ordering of sheets linked via
<meta> elements. Though I'm not quite sure, to be truthful, whether such sheets
should be treated exactly like ones linked via the header (that is, come before
all <link> and <style> sheets). Thoughts?
Blocks: 107567
I don't think that <meta> should act like <link>. <meta> should do the exact
same thing as adding the same header, that is IMHO the most logical behaviour. I
don't see any reason why <meta> should be as convinient and featurefull as
<link> unless if we absolutly have to do it.
WRT what should happen when you remove a <meta>. I don't really have a strong
oppinion either way. Should removing a <meta http-equiv="Refresh"> halt a
pending refresh? What about removing a <meta http-equiv="Content-Type">?
Comment 27•22 years ago
|
||
I'm of the opinion that <meta http-equiv="Link"> should work exactly the same
way as <link>. The insertion point should depend on the point in the HTTP stream
where the stylesheet is mentioned. <meta> should also be fully dynamically
editable, just like <link>.
This doesn't remove any of the functionality available to the author, indeed it
adds more functionality.
Comment 28•22 years ago
|
||
*** Bug 190839 has been marked as a duplicate of this bug. ***
Comment 29•22 years ago
|
||
OK. Could we decide on something here, please? Either we wontfix it, or we fix
it, but just having it sit here is silly.
Personally, I'm for making <meta> elements support nsIDOMLinkStyle and
nsIStyleSheetLinkingElement. I don't see any major problems this would cause.
I think ordering in document order would be more intuitive than ordering all the
<meta> sheets first. And that's the only possible issue I see with just fixing
this bug. Are there any others?
Updated•22 years ago
|
Priority: P3 → P2
IMHO a <meta http-equiv> should work *exactly* as adding the same http-header,
meaning that any linked stylesheets would be ordered as if the link came from a
http-header. Personally I think this is makes a lot of sense, although i'm
aware that i'm looking at it from a fairly technical point of view.
I'm also afraid that if we start adding interfaces to <meta> we will end up
overloading it a lot with time. And remember that if we give it those
interfaces it will have to have them even for elements like
<meta name="keywords" content="rubber ducks">. Giving functionality based on an
attribute-value makes for a lot of mess (look at the code for <input>), so
let's avoid it if we can.
That said, I won't go screaming in dispair if this is implemented anyway. I'm
just of the oppinion that we're better off without it :)
Comment 31•22 years ago
|
||
Yeah... the issue with interface proliferation is the same as with <link> -- it
just kinda sucks. :(
Updated•16 years ago
|
Whiteboard: [nsbeta2-]
Updated•15 years ago
|
QA Contact: ian → general
Comment 32•14 years ago
|
||
Note that per the current HTML5 draft, <meta http-equiv="Link"> sounds like it should do absolutely nothing.
Priority: P2 → --
Summary: [trk] (DOM2 StyleSheets) LinkStyle → Support LinkStyle and dynamic stylesheet changes on <meta http-equiv="Link">
Target Milestone: Future → ---
Updated•14 years ago
|
Assignee: bzbarsky → nobody
Depends on: 587928
Updated•14 years ago
|
QA Contact: general → style-system
Comment 33•6 years ago
|
||
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046
Move all DOM bugs that haven't been updated in more than 3 years and has no one currently assigned to P5.
If you have questions, please contact :mdaly.
Priority: -- → P5
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•