Skip to content

Commit

Permalink
Don't load PM icon in title unless topic is a PM
Browse files Browse the repository at this point in the history
  • Loading branch information
awesomerobot committed Jul 11, 2019
1 parent b848bd4 commit 1983f0d
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions app/assets/javascripts/discourse/templates/topic.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,15 @@
<h1 data-topic-id="{{unbound model.id}}">
{{#unless model.is_warning}}
{{#if siteSettings.enable_personal_messages}}
<a href={{pmPath}}>
<span class="private-message-glyph">{{d-icon "envelope"}}</span>
</a>
{{#if model.isPrivateMessage}}
<a href={{pmPath}}>
<span class="private-message-glyph">{{d-icon "envelope"}}</span>
</a>
{{/if}}
{{else}}
<span class="private-message-glyph">{{d-icon "envelope"}}</span>
{{#if model.isPrivateMessage}}
<span class="private-message-glyph">{{d-icon "envelope"}}</span>
{{/if}}
{{/if}}
{{/unless}}

Expand Down

1 comment on commit 1983f0d

@discoursebot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/lighthouse-accessibility-audit-issues/122612/4

Please sign in to comment.