IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-25 18:55:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

UX: fix tag & category clicks on mobile topic list (#20449)

Browse Source
This commit is contained in:
Kris 2023-02-24 15:29:57 -05:00 committed by GitHub
parent d3983d94a3
commit 90ed30b3ff
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/assets/javascripts/discourse/app/components/topic-list-item.js
View File

@ -260,9 +260,9 @@ export default Component.extend({
// make full row click target on mobile, due to size constraints // make full row click target on mobile, due to size constraints
if ( if (
this.site.mobileView && this.site.mobileView &&
(e.target.classList.contains("right") || e.target.matches(
e.target.classList.contains("topic-item-stats") || ".topic-list-data, .main-link, .right, .topic-item-stats, .topic-item-stats__category-tags, .discourse-tags"
e.target.classList.contains("main-link")) )
) { ) {
if (wantsNewWindow(e)) { if (wantsNewWindow(e)) {
return true; return true;

6
app/assets/stylesheets/mobile/topic-list.scss
View File

@ -189,9 +189,13 @@
.topic-item-stats__category-tags { .topic-item-stats__category-tags {
margin-right: 0.5em; margin-right: 0.5em;
.category,
.discourse-tags { .discourse-tags {
display: inline; display: inline;
}
.badge-wrapper,
.discourse-tag {
// disabling clicks because these targets are too small on mobile // disabling clicks because these targets are too small on mobile
pointer-events: none; pointer-events: none;
} }