Skip to content

Commit

Permalink
🐛 — [moderation] Show message content when clicking on its object
Browse files Browse the repository at this point in the history
The moderation help said "To read a message, click on its subject",
but it wasn’t true, you needed to click on the eye icon.
The eye icon it quite counter-intuitive, people has the reflex to
click on the subject of the mail to see it, like on mail clients.

It sounded better to me to make the interface like the documentation
instead of modifying the documentation.
  • Loading branch information
ldidry committed Aug 29, 2023
1 parent be2b8f6 commit 7681fad
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions default/web_tt2/modindex.tt2
Original file line number Diff line number Diff line change
Expand Up @@ -67,11 +67,15 @@
[%~ END %]
</div>
<div class="small-6 medium-3 columns" role="cell">
<a href="[% 'ajax/viewmod' | url_rel([list,msg.key]) %]"
data-reveal-id="mainviewmod" data-reveal-ajax="true"
data-tooltip aria-haspopup="true"
[% UNLESS msg.value.subject.length ~%]
<i>[%|loc%]No subject[%END%]</i>
[%~ ELSE ~%]
[% msg.value.subject %]
[%~ END %]
</a>
</div>
<div class="small-11 medium-2 columns" role="cell">
[% UNLESS msg.value.date ~%]
Expand Down

2 comments on commit 7681fad

@msokolo
Copy link

Choose a reason for hiding this comment

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

On line 72, the closing ">" of the A tag is missing.

@ldidry
Copy link
Contributor Author

@ldidry ldidry commented on 7681fad Jan 27, 2025

Choose a reason for hiding this comment

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

Indeed. Thank you, I created a pull request (#1934)

Please sign in to comment.