Skip to content

refactor: adopt hideCSS helper for eejsBlock_styles#81

Merged
JohnMcLear merged 1 commit intomainfrom
chore/adopt-hidecss-helper
May 10, 2026
Merged

refactor: adopt hideCSS helper for eejsBlock_styles#81
JohnMcLear merged 1 commit intomainfrom
chore/adopt-hidecss-helper

Conversation

@JohnMcLear
Copy link
Copy Markdown
Member

Replaces hand-rolled <style> tag concatenation with hideCSS(selector, {extra: ...}) from ep_plugin_helpers. The helper handles the display: none !important boilerplate; extra carries the layout rules needed once chat is hidden (#users height handling).

Wave 4 of the Phase 4 sweep: helper adoption for ep_disable_* plugins.

Replaces hand-rolled <style> tag concatenation with hideCSS() from
ep_plugin_helpers. The helper handles the boilerplate
`<style>${selector} { display: none !important; }</style>`
plus an opts.extra escape hatch for the non-hide rules
(#users height handling) needed once chat is suppressed.

Same runtime behavior — same CSS rules in the same <style> block.
@qodo-code-review
Copy link
Copy Markdown

ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one.

@JohnMcLear JohnMcLear merged commit 8ca15e0 into main May 10, 2026
3 checks passed
@JohnMcLear JohnMcLear deleted the chore/adopt-hidecss-helper branch May 10, 2026 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant