close
Skip to content

Hide sign out menu item using SVG selector#3788

Draft
pbcodesC wants to merge 1 commit intocode-charity:masterfrom
pbcodesC:master
Draft

Hide sign out menu item using SVG selector#3788
pbcodesC wants to merge 1 commit intocode-charity:masterfrom
pbcodesC:master

Conversation

@pbcodesC
Copy link
Copy Markdown

Hide the "Sign out" menu item using a CSS selector based on its icon (SVG path).

@ImprovedTube ImprovedTube marked this pull request as draft April 15, 2026 15:38
@ImprovedTube ImprovedTube added the Knowledge Base / Documentation for contributors We should repurpose this for future reference / Wiki / Education / Introduction label Apr 15, 2026
@ImprovedTube
Copy link
Copy Markdown
Member

ImprovedTube commented Apr 15, 2026

Thanks! (Please let me know, in which lines i may be talking to an LLM and/or a Human (no offense)). In pre-LLM times i would add the "integration" (condition, toggle and name) to this draft myself to be able to merge the PR and acknowledge a new contributor. Nowadays that will make less sense.

Technically, this PR is hiding the icon only yet and for every user, with no conditional toggle.
To hide a parent with CSS only, we need :has ( parent:has(child) {display:none} ). The English-based matching may stay to fall back to as well.



Practically, we can go further in our chosen(?) tasks, and for this one, allow to hide the whole menu as well and any item from the menu. (As a Sub-menu at Youtube: Appearance/Layout: Header : ...-Menu : ...-Button ). And Order those options by relevance or keep the original order but mark up bold whats relevant.


Further we can bring this feature to more usage - and save the many users one click by linking it on the visual map:
Image
( maybe in a more faint color still, since it is meant specifically to be noticed by eyes focusing on the top right of the map, by their own choice. But color is relative to the map. The whole map could have maximum contrast actually, since it is minimap afterall - and for long term settings too. )


rts mini map

(Soon we also may add the minimap on the front page to save yet another click. And we may open the front page as a browser sidebar (optional but by default for new users, so that accidental clicks outside do not close our Extension's sessions.)

@pbcodesC
Copy link
Copy Markdown
Author

My apologies, I am a human myself I may have relied on LLMs at times, but I do understand what you are trying to convey. As a newbie, I sometimes get confused or stuck, so I take help from AI to keep going
For now, I would like to keep this PR focused on hiding the sign-out menu item using a CSS-based approach. i understand your suggestions about extending this into a more flexible and configurable menu system,and will also update the selector to use a more robust parent-based approach (like :has) instead of relying only on text matching.
The UI/UX suggestions are helpful as well but i will keep things minimal for now and consider enhancements later.
thanks for the suggestions:)

@pbcodesC pbcodesC marked this pull request as ready for review April 16, 2026 07:23
@ImprovedTube ImprovedTube marked this pull request as draft April 16, 2026 16:15
@ImprovedTube
Copy link
Copy Markdown
Member

ImprovedTube commented Apr 16, 2026

Thank you! The directory menu is our GUI, the directory js&css contains the features.

# HIDE AI SUMMARY
------------------------------------------------------------------------------*/
html[it-hide-ai-summary='true'] ytd-expandable-metadata-renderer[has-video-summary],

You see the display:none features are listed by comma, only requiring the display:none once.
We sync all our toggles with attributes to the html root element (such as [it-hide-ai-summary='true'], which is possible prior to the rest of the DOM existing in the tab. - This is easy but efficient so i opened this repo: https://github.com/code-charity/display-none In one case such a 'display:none'-list (=set of features) called 'unHook' appears to be used by a million people: #2983 (comment) - Some people/developers add rules only for themselves in uBlockOrigin (which makes it convenient to pick the css selectors). (Yet that appears to not be common and done by less than 100 000 people in the world regularily. - All their effort should be crowd-sourced. More so since sometimes selectors require updates

And you can add your toggle in the end of menu skeleton (appearance, header), and the name in the top of the english locale file, so that i can merge this into our single version which is also running at all users'. - New code structures may be slow/personally annoying to start with. Many professionals will stay with one niche or a few / intersecting projects. Since each might be complex, yet not universal science, so it may be about productivity / quantity of use. So i might prefer the least verbose code and bundling series of similar tasks, since repetitive stuff is more pleasantly efficient. And we may want to serve the users efficiently. (For example chances are, unless one of us does semi-automatically extract more/all of youtube's SVG paths later in this thread, nobody might anytime soon, unwilling to start)

@ImprovedTube ImprovedTube force-pushed the master branch 4 times, most recently from d4783a9 to 0403173 Compare April 20, 2026 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Knowledge Base / Documentation for contributors We should repurpose this for future reference / Wiki / Education / Introduction

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants