Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-05 13:40:08 +01:00
Code Issues Projects Releases Packages Wiki Activity

Use id instead of name for matching

Browse source 
Signed-off-by: stevennovaryo <steven.novaryo@gmail.com>
This commit is contained in:
stevennovaryo 2025-05-15 15:04:59 +08:00
parent 3da5ef2cd4
commit a824b2781f
3 changed files with 29 additions and 18 deletions
Download patch file Download diff file Expand all files Collapse all files

8
components/layout/stylesheets/details.css
View file

@ -1,18 +1,18 @@
/* We should use `content-visibility` but it is yet to be implemented.
* These rule would not comply with ::details-content and should be removed
* with it's implementation */
slot:not([name]) {
slot[id="internal-contents-slot"] {
display: none;
}
:host([open]) slot:not([name]) {
:host([open]) slot[id="internal-contents-slot"] {
display: block;
}
summary[name=internal-fallback-summary] {
summary[id="internal-fallback-summary"] {
display: list-item;
counter-increment: list-item 0;
list-style: disclosure-closed inside;
}
:host([open]) summary[name=internal-fallback-summary] {
:host([open]) summary[id="internal-fallback-summary"] {
list-style-type: disclosure-open;
}

31
components/script/dom/htmldetailselement.rs
View file

@ -47,8 +47,6 @@ const DEFAULT_SUMMARY: &str = "Details";
struct ShadowTree {
summary: Dom<HTMLSlotElement>,
descendants: Dom<HTMLSlotElement>,
/// The summary that is displayed if no other summary exists
implicit_summary: Dom<HTMLElement>,
}
#[dom_struct]
@ -145,40 +143,45 @@ impl HTMLDetailsElement {
);
link_element.set_stylesheet(details_stylesheet.unwrap());
let summary = HTMLSlotElement::new(local_name!("slot"), None, &document, None, can_gc);
summary.upcast::<Element>().set_attribute(
&local_name!("name"),
AttrValue::from_atomic("internal-main-summary".to_owned()),
let summary_slot = HTMLSlotElement::new(local_name!("slot"), None, &document, None, can_gc);
summary_slot.upcast::<Element>().set_attribute(
&local_name!("id"),
AttrValue::from_atomic("internal-summary-slot".to_owned()),
can_gc,
);
root.upcast::<Node>()
.AppendChild(summary.upcast::<Node>(), can_gc)
.AppendChild(summary_slot.upcast::<Node>(), can_gc)
.unwrap();
let fallback_summary =
HTMLElement::new(local_name!("summary"), None, &document, None, can_gc);
fallback_summary.upcast::<Element>().set_attribute(
&local_name!("name"),
&local_name!("id"),
AttrValue::from_atomic("internal-fallback-summary".to_owned()),
can_gc,
);
fallback_summary
.upcast::<Node>()
.SetTextContent(Some(DEFAULT_SUMMARY.into()), can_gc);
summary
summary_slot
.upcast::<Node>()
.AppendChild(fallback_summary.upcast::<Node>(), can_gc)
.unwrap();
let descendants = HTMLSlotElement::new(local_name!("slot"), None, &document, None, can_gc);
let descendants_slot =
HTMLSlotElement::new(local_name!("slot"), None, &document, None, can_gc);
descendants_slot.upcast::<Element>().set_attribute(
&local_name!("id"),
AttrValue::from_atomic("internal-contents-slot".to_owned()),
can_gc,
);
root.upcast::<Node>()
.AppendChild(descendants.upcast::<Node>(), can_gc)
.AppendChild(descendants_slot.upcast::<Node>(), can_gc)
.unwrap();
let _ = self.shadow_tree.borrow_mut().insert(ShadowTree {
summary: summary.as_traced(),
descendants: descendants.as_traced(),
implicit_summary: fallback_summary.as_traced(),
summary: summary_slot.as_traced(),
descendants: descendants_slot.as_traced(),
});
self.upcast::<Node>()
.dirty(crate::dom::node::NodeDamage::OtherNodeDamage);

8
components/shared/script_layout/lib.rs
View file

@ -673,8 +673,16 @@ pub fn parse_ua_stylesheet(
.map(DocumentStyleSheet)
}
/// Parse stylesheet for <details> element to insert its UA shadow DOM.
///
/// TODO(stevennovaryo): The more approriate way to handle this is to use UA stylesheet.
/// But this element's styles needs to be in shadow-scoped stylesheet.
/// This could be done if an UA shadow-scoped UA sheet is introduced.
pub fn parse_details_stylesheet(
shared_lock: &SharedRwLock,
) -> Result<ServoArc<Stylesheet>, &'static str> {
// FIXME: We are parsing it as a Author stylesheet, but according to, it's nature
// it should be an user agent stylesheet. This is because we are only allowing
// the actual UA stylesheet to have that origin.
parse_stylesheet_as_origin(shared_lock, "details.css", DETAILS_CSS, Origin::Author)
}