Change HTML IDs for attribute macros and attributes

Rename `ItemType::BangMacro*` into `ItemType::DeclMacro*`
Rename `isBangMacro` field into `forceMacroHref`

Author: GuillaumeGomez

src/librustdoc/clean/types.rs

@@ -757,8 +757,8 @@ impl Item {
     pub(crate) fn types(&self) -> impl Iterator<Item = ItemType> {
         if let ItemKind::MacroItem(_, macro_kinds) = self.kind {
             Either::Right(macro_kinds.iter().map(|kind| match kind {
-                MacroKinds::ATTR => ItemType::BangMacroAttribute,
-                MacroKinds::DERIVE => ItemType::BangMacroDerive,
+                MacroKinds::ATTR => ItemType::DeclMacroAttribute,
+                MacroKinds::DERIVE => ItemType::DeclMacroDerive,
                 MacroKinds::BANG => ItemType::Macro,
                 _ => panic!("unsupported macro kind {kind:?}"),
             }))

src/librustdoc/formats/item_type.rs

@@ -105,8 +105,8 @@ item_type! {
     // distinction because they will point to a `macro.[name].html` file and not
     // `[attr|derive].[name].html` file, so the link generation needs to take it into account while
     // still having the filtering working as expected.
-    BangMacroAttribute = 28,
-    BangMacroDerive = 29,
+    DeclMacroAttribute = 28,
+    DeclMacroDerive = 29,
 }
 
 impl<'a> From<&'a clean::Item> for ItemType {
@@ -226,8 +226,8 @@ impl ItemType {
             ItemType::AssocConst => "associatedconstant",
             ItemType::ForeignType => "foreigntype",
             ItemType::Keyword => "keyword",
-            ItemType::ProcAttribute | ItemType::BangMacroAttribute => "attr",
-            ItemType::ProcDerive | ItemType::BangMacroDerive => "derive",
+            ItemType::ProcAttribute | ItemType::DeclMacroAttribute => "attr",
+            ItemType::ProcDerive | ItemType::DeclMacroDerive => "derive",
             ItemType::TraitAlias => "traitalias",
             ItemType::Attribute => "attribute",
         }

src/librustdoc/html/render/mod.rs

@@ -543,10 +543,10 @@ impl AllTypes {
             ItemType::TypeAlias => self.type_aliases.insert(ItemEntry::new(new_url, name)),
             ItemType::Static => self.statics.insert(ItemEntry::new(new_url, name)),
             ItemType::Constant => self.constants.insert(ItemEntry::new(new_url, name)),
-            ItemType::ProcAttribute | ItemType::BangMacroAttribute => {
+            ItemType::ProcAttribute | ItemType::DeclMacroAttribute => {
                 self.attribute_macros.insert(ItemEntry::new(new_url, name))
             }
-            ItemType::ProcDerive | ItemType::BangMacroDerive => {
+            ItemType::ProcDerive | ItemType::DeclMacroDerive => {
                 self.derive_macros.insert(ItemEntry::new(new_url, name))
             }
             ItemType::TraitAlias => self.trait_aliases.insert(ItemEntry::new(new_url, name)),
@@ -2605,8 +2605,8 @@ impl ItemSection {
             Self::AssociatedConstants => "associated-consts",
             Self::ForeignTypes => "foreign-types",
             Self::Keywords => "keywords",
-            Self::Attributes => "attributes",
-            Self::AttributeMacros => "attribute-macros",
+            Self::Attributes => "attribute-docs",
+            Self::AttributeMacros => "attributes",
             Self::DeriveMacros => "derives",
             Self::TraitAliases => "trait-aliases",
         }
@@ -2667,8 +2667,8 @@ fn item_ty_to_section(ty: ItemType) -> ItemSection {
         ItemType::ForeignType => ItemSection::ForeignTypes,
         ItemType::Keyword => ItemSection::Keywords,
         ItemType::Attribute => ItemSection::Attributes,
-        ItemType::ProcAttribute | ItemType::BangMacroAttribute => ItemSection::AttributeMacros,
-        ItemType::ProcDerive | ItemType::BangMacroDerive => ItemSection::DeriveMacros,
+        ItemType::ProcAttribute | ItemType::DeclMacroAttribute => ItemSection::AttributeMacros,
+        ItemType::ProcDerive | ItemType::DeclMacroDerive => ItemSection::DeriveMacros,
         ItemType::TraitAlias => ItemSection::TraitAliases,
     }
 }

src/librustdoc/html/render/print_item.rs

@@ -235,8 +235,8 @@ fn item_module(cx: &Context<'_>, item: &clean::Item, items: &[clean::Item]) -> i
             // we cheat by turning them into their "proc-macro equivalent".
             for type_ in item.types() {
                 let type_ = match type_ {
-                    ItemType::BangMacroAttribute => ItemType::ProcAttribute,
-                    ItemType::BangMacroDerive => ItemType::ProcDerive,
+                    ItemType::DeclMacroAttribute => ItemType::ProcAttribute,
+                    ItemType::DeclMacroDerive => ItemType::ProcDerive,
                     type_ => type_,
                 };
                 not_stripped_items.entry(type_).or_default().push((index, item));

src/librustdoc/html/static/js/main.js

@@ -793,8 +793,8 @@ function preLoadCss(cssUrl) {
             //block("associatedconstant", "associated-consts", "Associated Constants");
             block("foreigntype", "foreign-types", "Foreign Types");
             block("keyword", "keywords", "Keywords");
-            block("attribute", "attributes", "Attributes");
-            block("attr", "attribute-macros", "Attribute Macros");
+            block("attribute", "attribute-docs", "Attributes");
+            block("attr", "attributes", "Attribute Macros");
             block("derive", "derives", "Derive Macros");
             block("traitalias", "trait-aliases", "Trait Aliases");
         }

src/librustdoc/html/static/js/rustdoc.d.ts

@@ -265,7 +265,7 @@ declare namespace rustdoc {
          * But the documentation lives in a single `macro.NAME.html` page, and
          * this boolean flag is used for generating that HREF.
          */
-        isBangMacro: boolean,
+        forceMacroHref: boolean,
     }
 
     /**

src/librustdoc/html/static/js/search.js

@@ -1658,13 +1658,13 @@ class DocSearch {
             deprecated: raw[6] === 1 ? true : false,
             unstable: raw[7] === 1 ? true : false,
             associatedItemDisambiguatorOrExternCrateUrl: raw.length === 8 ? null : raw[8],
-            isBangMacro: false,
+            forceMacroHref: false,
         };
         if (item.ty === 28 || item.ty === 29) {
             // "proc attribute" is 23, "proc derive" is 24 whereas "bang macro attribute" is 28 and
             // "bang macro derive" is 29, so 5 of difference to go from the latter to the former.
             item.ty -= 5;
-            item.isBangMacro = true;
+            item.forceMacroHref = true;
         }
         return item;
     }
@@ -2164,7 +2164,7 @@ class DocSearch {
             let displayPath;
             let href;
             let traitPath = null;
-            const type = item.entry && item.entry.isBangMacro ? "macro" : itemTypesName[item.ty];
+            const type = item.entry && item.entry.forceMacroHref ? "macro" : itemTypesName[item.ty];
             const name = item.name;
             let path = item.modulePath;
             let exactPath = item.exactModulePath;

src/librustdoc/json/conversions.rs

@@ -899,8 +899,8 @@ impl FromClean<ItemType> for ItemKind {
             Keyword => ItemKind::Keyword,
             Attribute => ItemKind::Attribute,
             TraitAlias => ItemKind::TraitAlias,
-            ProcAttribute | BangMacroAttribute => ItemKind::ProcAttribute,
-            ProcDerive | BangMacroDerive => ItemKind::ProcDerive,
+            ProcAttribute | DeclMacroAttribute => ItemKind::ProcAttribute,
+            ProcDerive | DeclMacroDerive => ItemKind::ProcDerive,
         }
     }
 }

tests/rustdoc-gui/attr-macros.goml

@@ -53,7 +53,7 @@ define-function: (
 
 // Now we check it's correctly listed in the crate page.
 go-to: "file://" + |DOC_PATH| + "/test_docs/index.html"
-call-function: ("crate_page", {"name": "attr_macro", "section_id": "attribute-macros"})
+call-function: ("crate_page", {"name": "attr_macro", "section_id": "attributes"})
 call-function: ("crate_page", {"name": "derive_macro", "section_id": "derives"})
 // We also check we don't have duplicated sections.
 assert-count: ("//*[@id='main-content']/h2[text()='Attribute Macros']", 1)
@@ -77,7 +77,7 @@ define-function: (
 
 // And finally we check it's correctly listed in the "all items" page.
 go-to: "file://" + |DOC_PATH| + "/test_docs/all.html"
-call-function: ("all_items_page", {"name": "attr_macro", "section_id": "attribute-macros"})
+call-function: ("all_items_page", {"name": "attr_macro", "section_id": "attributes"})
 call-function: ("all_items_page", {"name": "derive_macro", "section_id": "derives"})
 
 // We now check a macro with all 3 different kinds.

tests/rustdoc-gui/module-items-font.goml

@@ -67,19 +67,19 @@ assert-css: (
 )
 // attributes
 assert-css: (
-    "#attributes + .item-table dt a",
+    "#attribute-docs + .item-table dt a",
     {"font-family": '"Fira Sans", Arial, NanumBarunGothic, sans-serif'},
 )
 assert-css: (
-    "#attributes + .item-table dd",
+    "#attribute-docs + .item-table dd",
     {"font-family": '"Source Serif 4", NanumBarunGothic, serif'},
 )
 // attribute macros
 assert-css: (
-    "#attribute-macros + .item-table dt a",
+    "#attributes + .item-table dt a",
     {"font-family": '"Fira Sans", Arial, NanumBarunGothic, sans-serif'},
 )
 assert-css: (
-    "#attribute-macros + .item-table dd",
+    "#attributes + .item-table dd",
     {"font-family": '"Source Serif 4", NanumBarunGothic, serif'},
 )