mirror of
https://github.com/discourse/discourse.git
synced 2024-11-22 08:57:10 -06:00
UX: Move "watching topic" user-tip to the other button (#29323)
Moves the user-tip from the topic-timeline notifications button to the one at the bottom of the topic page. Three reasons: 1. new users are more likely to use the button that has the full text (and description) rather than the icon-only one 2. we hide the timeline button when scrolled all the way to the bottom of the page, and then the tip doesn't seems to be attached to anything 3. we might be removing the timeline button altogether in the near future
This commit is contained in:
parent
f45559a8b3
commit
b2167f7651
@ -91,17 +91,20 @@
|
||||
<PinnedButton @pinned={{this.topic.pinned}} @topic={{this.topic}} />
|
||||
|
||||
{{#if this.showNotificationsButton}}
|
||||
<TopicNotificationsButton
|
||||
@topic={{this.topic}}
|
||||
class="notifications-button-footer"
|
||||
/>
|
||||
|
||||
{{#if this.showNotificationUserTip}}
|
||||
<UserTip
|
||||
@id="topic_notification_levels"
|
||||
@triggerSelector=".notifications-button"
|
||||
@triggerSelector=".notifications-button-footer details"
|
||||
@titleText={{i18n "user_tips.topic_notification_levels.title"}}
|
||||
@contentText={{i18n "user_tips.topic_notification_levels.content"}}
|
||||
@priority={{800}}
|
||||
/>
|
||||
{{/if}}
|
||||
|
||||
<TopicNotificationsButton @topic={{this.topic}} />
|
||||
{{/if}}
|
||||
|
||||
<PluginOutlet
|
||||
|
@ -113,7 +113,7 @@ export default class TopicNotificationsButton extends Component {
|
||||
}
|
||||
|
||||
<template>
|
||||
<div class="topic-notifications-button">
|
||||
<div class="topic-notifications-button" ...attributes>
|
||||
{{#if this.appendReason}}
|
||||
<p class="reason">
|
||||
<TopicNotificationsOptions
|
||||
|
Loading…
Reference in New Issue
Block a user