jenkins-bot has submitted this change and it was merged.
Change subject: Apply the nomobile class
......................................................................
Apply the nomobile class
The ext.flaggedRevs.basic is not currently targeted at mobile and looking
at it, there is no need for it to be.
Unfortunately this means the banner in FlaggablePageview is visible in mobile
browsers taking up real estate.
This generic solution allows the hiding of this banner at mobile resolutions
Bug: T119344
Change-Id: I8c718394f870812ec946dffb2b89474757c64b08
---
M frontend/FlaggablePageView.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Aaron Schulz: Looks good to me, approved
jenkins-bot: Verified
diff --git a/frontend/FlaggablePageView.php b/frontend/FlaggablePageView.php
index 5eb51a5..6746776 100755
--- a/frontend/FlaggablePageView.php
+++ b/frontend/FlaggablePageView.php
@@ -380,7 +380,7 @@
}
# Wrap tag contents in a div
if ( $tag != '' ) {
- $css = "{$tagClass} plainlinks noprint";
+ $css = "{$tagClass} plainlinks noprint nomobile";
$notice = "<div id=\"mw-fr-revisiontag\"
class=\"{$css}\">{$tag}</div>\n";
$this->reviewNotice .= $notice;
}
--
To view, visit https://gerrit.wikimedia.org/r/271424
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I8c718394f870812ec946dffb2b89474757c64b08
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/FlaggedRevs
Gerrit-Branch: master
Gerrit-Owner: Jdlrobson <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Jackmcbarn <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
