While evaluating this module, I thought the UI could use a little cleanup to match and align to the other UI messages on the ADmin menu.

Patch to follow, thanks!

File to edit:
zt_megamenu.links.menu.yml

Configure the zt_megamenu sub-menu contents.
Should read
Configure the ZT Megamenu sub-menu contents.

ZT-Megamenu Settings Form
should read
ZT Megamenu Settings Form

CommentFileSizeAuthor
#2 zt_megamenu-ui-cleanup-2981580-2.patch504 bytesvolkswagenchick

Comments

volkswagenchick created an issue. See original summary.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented
Assigned: volkswagenchick » Unassigned
Status: Active » Needs review
StatusFileSize
new zt_megamenu-ui-cleanup-2981580-2.patch504 bytes

I uploaded a patch that cleans up the UI a bit. Thanks for your work on this project!

alonaoneill’s picture

alonaoneill commented
Status: Needs review » Reviewed & tested by the community

Reviewed the patch for spelling and grammar. Formatting aligns to Drupal documentation standards.
Marking RTBC. Thanks for your work on a module.

binnythomas’s picture

binnythomas commented
Status: Reviewed & tested by the community » Fixed

We have applied the patch. Thanks everyone. Hope you found the module useful.

volkswagenchick’s picture

volkswagenchick
she/her
Primary language English
Location San Francisco Bay Area
commented

Thankyou for all the work you do!!
I am encouraging team mates to help out on Drupal.org, could you go back and give aAlona O'neill (alonaoneill) a credit for her review? She took the time to test and make sure everything was correct.
The box next to her name just gets ticked and she gets a credit! I see it as a way of incentive to help get involved it the community! It takes a village.

Cheers!

AmyJune - volkswagenchick

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

binnythomas’s picture

binnythomas commented