Closed (fixed)
Project:
Sitemap
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
30 Mar 2021 at 10:59 UTC
Updated:
24 Mar 2022 at 16:49 UTC
Jump to comment: Most recent, Most recent file


Comments
Comment #2
baroneHere's the patch that fixes the menu label issue.
Comment #3
baroneComment #5
baroneComment #6
monojithalder commentedComment #8
gngn commentedThe patch in #6 renders an empty h2 title (if we set it empty in the config).
That is because of the ' ' (string with one blank) in Menu::view().
I see no reason for this (and I think empty h2 tags are invalid HTML).
So I propose to use '' instead of ' '.
Comment #12
dmitriy.trt commentedCreated MR !8 with the following alternative approach:
It's better to use changes of this MR together with MR !7 of #3231515: Save settings of enabled plugins only, because otherwise default titles only appear once on the module installation.
Comment #14
akalata commentedThanks @barone for the detailed write-up, everyone else for keeping this thread going, and to @dmitriy.trt for keeping the default values when a plugin is enabled.