2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2026-03-03 23:54:20 +08:00
discourse/themes/horizon/spec/system
Régis Hanol 1499ab0741
FIX: Pass tag name instead of id to composer in Horizon sidebar button (#38112)
The sidebar "New Topic" button in Horizon was passing `this.tag?.id` (a
numeric ID) to `composer.openNewTopic()`, which expects a
comma-separated string of tag names. This caused two issues:

- For non-staff users, `filterTags()` tried to access `.content` on a
number, throwing "Cannot read properties of undefined (reading
'filter')" and preventing the composer from opening entirely.
- For staff users, `filterTags()` passed the number through, resulting
in the numeric ID being displayed as the tag name in the composer.

Switching to `this.tag?.name` aligns with the pattern used by the
discovery list controller and the composer's expected API.

Also adds a system spec covering the tag page → new topic flow.

Ref - t/178629
2026-02-27 14:46:37 +01:00
..
page_objects/components DEV: Add a computed style matcher (#36381) 2025-12-02 15:14:08 +01:00
.gitkeep FIX: import Horizon system theme (#33229) 2025-06-17 14:03:38 +08:00
composer_peek_spec.rb DEV: add shortcut fab!(:variable, :fabricator) to specs (#33577) 2025-07-11 11:16:34 -03:00
core_features_spec.rb FIX: import Horizon system theme (#33229) 2025-06-17 14:03:38 +08:00
horizon_high_level_spec.rb DEV: Clean up requires (#34946) 2025-09-24 12:00:22 +02:00
sidebar_topic_button_spec.rb FIX: Pass tag name instead of id to composer in Horizon sidebar button (#38112) 2026-02-27 14:46:37 +01:00
user_color_palette_selector_spec.rb DEV: Fix random typos (#36505) 2025-12-08 11:33:19 +01:00