search

Charcoal-SE / userscripts

Collection of userscripts that are used by/are useful to Charcoal.
https://charcoal-se.org/scripts
Apache License 2.0
28 stars 27 forks source link

Update to match SE changes to post quick-links #165

Closed codygray closed 2 years ago

codygray commented 3 years ago

Modifies the script's parsing and manipulation of the DOM to comply with Stack Exchange's recent changes to the post quick-links menu items. These changes were not announced, but have since been discussed on Meta (https://meta.stackexchange.com/questions/359477/capitalization-of-post-action-links-is-causing-multiple-rows-of-links).

makyen commented 2 years ago

Thanks for this. However, I'm going to close this for now, as the primary functional changes were made in a different sequence of edits. The DOM changes in the .post-menu were a little bit more extensive in order to match the structure which SE is using at this point. I don't recall if that was a second change on SE's part, or if that changed all at one time.

This does remove some cruft in the code, .post-menu-container, which is useful to remove, but I'd want to double-check that it wasn't still being used on the NATO page, with and without NATO Enhancements, or in any of the other userscripts this was attempting to be compatible with. Testing those pages was pushed to later as it was more work than I wanted to do. Some of the userscripts with which this is/was attempting to be compatible with may be creating the .post-menu structure themselves. I think they were all updated past SE's .post-menu-container phase, but I would need to double check.