diff --git a/doc/PLUGIN_FEATURE_DISABLES.md b/doc/PLUGIN_FEATURE_DISABLES.md index 41c87c46a..f82dcd232 100644 --- a/doc/PLUGIN_FEATURE_DISABLES.md +++ b/doc/PLUGIN_FEATURE_DISABLES.md @@ -26,6 +26,8 @@ Tags currently in use: - `@feature:username` - `@feature:clear-authorship` - `@feature:error-gritter` +- `@feature:line-numbers` +- `@feature:rtl-toggle` ### 2. A plugin declares the features it disables in its `ep.json` diff --git a/src/tests/frontend-new/specs/chat.spec.ts b/src/tests/frontend-new/specs/chat.spec.ts index bf3e2f382..bdabd4932 100644 --- a/src/tests/frontend-new/specs/chat.spec.ts +++ b/src/tests/frontend-new/specs/chat.spec.ts @@ -150,7 +150,12 @@ test('chat icon click reveals chatbox after a disable → enable cycle', { // Title-bar layout / glyph regressions from #7590 review. test('chat title bar lays out as a centred flex row with underscore minimize', { - tag: '@feature:chat', + // `@feature:rtl-toggle` because the symmetric leftGap/rightGap + // assertion is only valid in LTR — the colibris #titlebar padding + // rule ships a one-sided pad that flips under `body[dir=rtl]`, + // throwing the gap apart by ~170px. A plugin that forces RTL on + // (e.g. ep_right_to_left) declares this in its disables list. + tag: ['@feature:chat', '@feature:rtl-toggle'], }, async ({page}) => { await showChat(page); diff --git a/src/tests/frontend-new/specs/rtl_url_param.spec.ts b/src/tests/frontend-new/specs/rtl_url_param.spec.ts index a279dc6e4..f094da101 100644 --- a/src/tests/frontend-new/specs/rtl_url_param.spec.ts +++ b/src/tests/frontend-new/specs/rtl_url_param.spec.ts @@ -13,7 +13,7 @@ test.describe('RTL URL parameter', function () { await expect(page.locator('#options-rtlcheck')).toBeChecked(); }); - test('rtl=false disables RTL mode after rtl=true', async function ({page}) { + test('rtl=false disables RTL mode after rtl=true', {tag: '@feature:rtl-toggle'}, async function ({page}) { // First enable RTL via URL await appendQueryParams(page, {rtl: 'true'}); await expect(page.locator('#options-rtlcheck')).toBeChecked(); @@ -23,7 +23,7 @@ test.describe('RTL URL parameter', function () { await expect(page.locator('#options-rtlcheck')).not.toBeChecked(); }); - test('no rtl param falls back to the pad setting after an RTL URL override', async function ({page}) { + test('no rtl param falls back to the pad setting after an RTL URL override', {tag: '@feature:rtl-toggle'}, async function ({page}) { // Enable RTL via URL for the current page load only await appendQueryParams(page, {rtl: 'true'}); await expect(page.locator('#options-rtlcheck')).toBeChecked();