DEV: removes caret from notifications-tracking on mobile (#30369)

We never want to show this caret on mobile.
This commit is contained in:
Joffrey JAFFEUX 2024-12-19 12:06:02 +01:00 committed by GitHub
parent 9ca8f706bf
commit 1f39ce87a5
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 23 additions and 2 deletions

View File

@ -1,6 +1,7 @@
import Component from "@glimmer/component";
import { fn, hash } from "@ember/helper";
import { action } from "@ember/object";
import { service } from "@ember/service";
import DButton from "discourse/components/d-button";
import DropdownMenu from "discourse/components/dropdown-menu";
import PluginOutlet from "discourse/components/plugin-outlet";
@ -21,12 +22,14 @@ function constructKey(prefix, level, suffix, key) {
}
class NotificationsTrackingTrigger extends Component {
@service site;
get showFullTitle() {
return this.args.showFullTitle ?? true;
}
get showCaret() {
return this.args.showCaret ?? true;
return this.site.desktopView && (this.args.showCaret ?? true);
}
get title() {
@ -48,7 +51,7 @@ class NotificationsTrackingTrigger extends Component {
{{/if}}
{{#if this.showCaret}}
{{icon "angle-down"}}
{{icon "angle-down" class="notifications-tracking-btn__caret"}}
{{/if}}
</template>
}

View File

@ -78,4 +78,22 @@ module("Integration | Component | TopicTracking", function (hooks) {
);
});
});
test("caret", async function (assert) {
await render(<template><TopicNotificationsTracking /></template>);
assert.dom(".notifications-tracking-btn__caret").exists();
});
});
module("Integration | Component | TopicTracking - Mobile", function (hooks) {
setupRenderingTest(hooks);
test("no caret", async function (assert) {
this.site.desktopView = false;
await render(<template><TopicNotificationsTracking /></template>);
assert.dom(".notifications-tracking-btn__caret").doesNotExist();
});
});