Commit graph

362 commits

Author SHA1 Message Date
Saint Johann 51ac315a57 Fix notification z-index in Minerva
If you do not match the default value (10000), overlays display under OOUI popups.

Bug: T356513
Change-Id: I31cbd6c68652b1cb34c8dc589b1d3bc2ba750328
2024-11-06 16:15:59 +00:00
jenkins-bot 67136c0350 Merge "Partial Revert "Make sure contributor's name is on its line"" 2024-10-28 05:04:50 +00:00
Jdlrobson b130c93b8f Partial Revert "Make sure contributor's name is on its line"
This partially reverts commit f8d410fb98.
and replaces the selector with a selector that considers spans which have link inside them. This ensures the whole row remains clickable
and takes the user to the diff.

Reason for revert: Caused
https://en.wikipedia.org/wiki/Wikipedia:Village_pump_(technical)#Recent_changes_on_mobile

Bug: T378142
Change-Id: Iff944817d9fe1664a60ec30413c159247706a5e7
2024-10-27 21:35:04 +00:00
matr1x-101 29219736a6 Allow IP information popup to overlay on top of "compare selected versions" box
Set bottom "compare selected versions" box to use z-index zero so IP information popup goes over "compare selected versions" box

Bug: T304944
Change-Id: Id78e643f73dbba740501c67912c869621786a4b3
2024-10-23 19:56:18 +00:00
Ebrahim Byagowi f8d410fb98 Make sure contributor's name is on its line
There was an overlooked usecase in T377503 which regressed
by change of hidden characters in the core, this change
makes it possible to remove the last hidden character
there in I3f08595d654fbfe18627b44839eaa5c9fd38274c
and fix that overlooked usecase.

Bug: T377503
Change-Id: Ifb458f80f08d9c6f717f4c035c7d5120cccb04cf
2024-10-21 23:26:41 +00:00
Jon Robson ca232770db Strengthen CSS selectors for Minerva Echo icons
The .minerva-icon rule sets a transparent PNG. If rules are loaded
before (as in the case of Echo) they get overriden. Make these rules
more specific so they do not apply.

Bug: T376491
Change-Id: Ia5399fa500df69b0a0d34931e30d961fe62a2124
2024-10-07 18:16:40 +00:00
Jon Robson 83c0105418 Fixes issues with links flagged by Pixel
Bug: T375832
Follow up to
    I6e9e29fd5b140ae8eb4e6b011d558ea778c92028
Change-Id: I3f52f88b2b08f502eb584af48dcde0c5a8eb6674

Change-Id: Ib6a5de1397f2df8dab68deb97f079b316d4701b3
2024-09-28 01:24:10 +00:00
Abhii67534 9a2ee050ed Fix alignment(Minerva):current marker was not aligned with message
Bug: T372392
Change-Id: I6c00e78fec0dada22134a72ca8e0b6f50764bd13
2024-09-10 22:16:21 +00:00
MusikAnimal 03948a7a04 Explicitly set font size in VisualEditor + CodeMirror 6
This only effects Minerva desktop users of VE + CodeMirror.

Bug: T357482
Change-Id: I2569517ba2ce67d3d6ac486ddbfb9a3ce99d8c8c
2024-08-20 17:30:30 -04:00
Dr4goniez 8a353c5421 Remove !important from special page heading styles
The !important-less version still overrides 8ce4955ada/resources/skins.minerva.base.styles/content/headings.less (73).

Bug: T296598
Change-Id: I894b2e816b4ca739b4ba7cea53cdef7a85b1b776
2024-08-16 21:52:01 +00:00
Jdrewniak 83e7a06a52 Prevent dark-mode styles from affecting print media
Prevents dark-mode styles from affecting print media
by adding `@media screen` to dark-mode related styles.

Bug: T371331
Change-Id: Ieeca30d086d8b8705aba96441a576e3d8e127838
2024-08-08 14:27:30 -04:00
libraryupgrader 842a91590a build: Updating npm dependencies
* eslint-config-wikimedia: 0.27.0 → 0.28.2
  The following rules are failing and were disabled:
  * tests/selenium:
    * implicit-arrow-linebreak
    * no-mixed-spaces-and-tabs
* grunt-banana-checker: 0.11.1 → 0.13.0
* stylelint-config-wikimedia: 0.16.1 → 0.17.2
  The following rules no longer exist and were removed:
  * stylistic/selector-list-comma-newline-after
* braces: 3.0.2 → 3.0.3
  * https://github.com/advisories/GHSA-grv7-fg5c-xmjg

Change-Id: Ia94454c1da778f241085714e1601a0233d547570
2024-08-01 15:27:33 +01:00
Taavi Väänänen a01e0042c3 Fix mobile diff author note on dark mode
Updates the styles for the author note shown at the bottom of the screen
on the diff page on narrow displays to use Codex design tokens so that
it will use dark colors when dark mode is enabled.

The color for the edit count is changed from #72777d (color-placeholder
/ color-disabled) to a darker color #54595d to use a more appropriate
token (color-subtle).

Bug: T370003
Change-Id: I3340e8794036a1c7cd8aa4cfebd2ecfb50d2cbcf
2024-07-23 21:57:13 +00:00
Jdlrobson 400546e56f Partial revert "Temporary fix for enabling Mobile VisualEditor in night mode"
This reverts commit 1a4a16062c.

Reason for revert: Fixes were upstreamed.
Depends-On: Ia35231be8a0325671f329fbd71a72da826505587
Bug: T368116
Change-Id: Ia941791b59e21d0c9191923ea9aa19842ebf787c
2024-07-02 17:44:24 +00:00
Jon Robson 1a4a16062c Temporary fix for enabling Mobile VisualEditor in night mode
Bug: T366832
Change-Id: If03069339261e380dc49defedccb01830d162483
2024-06-20 16:36:51 -04:00
Jon Robson 72bcebeba8 Use upstream codex variables for diffs and red links
Bug: T361717
Bug: T363743
Change-Id: I4e49a07f9f092ab00e2b2d12fcb146c91a21d8d8
2024-06-12 10:16:59 -07:00
jenkins-bot 91fd8a977b Merge "Improve Minerva drop-shadows in dark-mode" 2024-06-12 16:02:29 +00:00
Jan Drewniak 98eaa00cd5 Improve Minerva drop-shadows in dark-mode
Converts several drop-shadows to use standard Codex values:
- `.search-box .search` uses `@box-shadow-drop-small`
- `.drawer, .toast` uses `@box-shadow-drop-xx-large`
- `.toggle-list__list--drop-down` uses `@border-color-base`

Bug: T364425
Change-Id: Id00e525655d3a7138aa65f424fd6ab2ff82290c8
2024-06-12 04:05:30 +00:00
Steph Toyofuku 6f98cc6460 (Almost) no more CSS variables in minerva
Remove direct usage of CSS variables in Minerva, and replace them with
codex design tokens again.  Document this decision in the original ADR

Note: there are still a small number of CSS variables in use, including
most notably the --color-link-red fix, which broke when I removed it,
but this change takes care of all the ones that could be easily replaced

Bug: T363743
Change-Id: I7d3a9dceb908167078987de1733774c8bd4bea2f
2024-06-11 11:44:47 -07:00
Volker E dd154b4ea7 styles: Replace deprecated breakpoints
The values have slightly changed:
- Tablet breakpoint is decreased by 80px,
  pushing lesser devices into a mobile only experience but
  should make barely a difference to vast majority of our users.
- Biggest difference is in `min-width-breakpoint-desktop`,
  which is increased by 120px in comparison to
  `@width-breakpoint-desktop: 1000px`

Also removing one out-dated feature phone clause.

TECHNICAL CHANGES:
* The previous breakpoint was tied to the maximum content width.
The previous maximum content width is preserved, and several media
queries are adjusted to make use of it.
* The header (logo) is not optimized for  640px-720px. For now
a temporary fix is but in place which will be investigated as part
of T366859

VISUAL CHANGES:
* 6 visual changes which improve alignment in language and
visual editor overlay and diffs (where wikidiff2 is not
installed)

Bug: T349793
Depends-On: I3afba8c51f60de9271054499bfa3ffbcc1a9d779
Change-Id: I9552d8ad7509aae90e15edda26e786465773d3ac
2024-06-07 12:52:39 -07:00
bwang 179d48d4f8 Fix color contrast on toast/drawers
Bug: T364117
Change-Id: Ibdb5cd49f8d4fcd1f69bc1209789a936ba8067ec
2024-05-29 05:46:46 +00:00
Jon Robson 5643b8c2b8 Drop responsive behaviour
This is not working as expected - when I view the watchlist
it looks misaligned.

At this stage it is not vital to keep this view identical
with the existing version.

Bug: T109277
Change-Id: I2768a83cdd015867c9d66ded63acfe369629cf52
2024-05-17 16:06:32 +00:00
Steph Toyofuku b7e54644b7 Make language selector disabled state accessible contrast
Remove the opacity filter and indicate a disabled state using the
disabled color for both text and the icon.  Add a new disabled icon
variant to accomplish this

Also fixes a small issue where the toast notification was unreadable in
night mode due to using --color-inverted.  Switch this to
--color-inverted-fixed, which was added for this purpose

Visual change for the language selector button, when there are no other
languages available

Bug: T361693
Change-Id: Id2f80028d967bf644673f651bb89718fa8723d22
2024-05-13 22:13:33 +00:00
Jon Robson 8239b8bd9c Style Special:NewPage consistently with Special:RecentChanges
Bug: T360843
Change-Id: Id75c7036cdff7f0a597fd58857fb367b16fffc79
2024-05-10 14:51:17 +00:00
jenkins-bot 520fa3a671 Merge "Remove old styles for CAPTCHA on Special:CreateAccount" 2024-05-04 16:08:46 +00:00
Sjoerd de Bruin 40c819c0c3 Apply white-space:normal to buttons in preferences
Bug: T356467
Change-Id: Iffc24107c45fdf078129cbd74eb03b1841bdb953
2024-05-04 12:49:55 +00:00
Bartosz Dziewoński 690c6aafe8 Remove old styles for CAPTCHA on Special:CreateAccount
I think they custom design is no longer needed now that the form uses
Codex, and some of the rules are applying incorrectly now and causing
bad rendering.

Depends-On: I3dd4509e8fbe43c8c3fee405d81e1e6efe5ee296
Bug: T364214
Change-Id: I994c1dc6492c39964b21f0ab2e8a0e0ca848cc7a
2024-05-04 13:05:03 +02:00
Jon Robson 1f83fd0d68 History rows should be clickable
Bug: T362683
Change-Id: I8a3b3554991eb8229b0d734fd577c48868480fdc
2024-05-02 12:08:39 +00:00
Jon Robson ba64c4168d Complete transition to Codex color palettes
* Update mediawiki.skin.variables to point non-Codex tokens to
Codex token equivalents
* Replace box-shadow-color-drawer with @box-shadow-color-base
* Leave FIXME comments for migrating the last two CSS variables
* Replace direct access to CSS variable with @color-link-new
design token instead since the former may no longer exist.

Bug: T360845
Change-Id: Ie531044d320cde50689064f10ff00db5adcaf859
2024-04-29 14:16:36 -07:00
Jon Robson 9ad8766d67 AMC-disabled Watchlist should mirror MobileFrontend version
* Hide RC filters on AMC -disabled view
* Add margin between tabs and list and full width border
* Hide 3rd and 4th tabs on watchlist view

Bug: T266065
Change-Id: I39a20bb763c6057d65550bdfface0d9156799dac
2024-04-25 23:29:36 +00:00
Jon Robson db5852ecc7 Remove styles for deleted diff special page
MobileDiff special page no longer exists so none of this CSS which applies to #mw-mf prefixed code is not applying to anything.

Bug: T360389
Change-Id: I6638850a6e1cd1e27072ecd17dd1524f62f976cd
2024-04-25 19:07:12 +00:00
Jon Robson 32bf846613 Apply history styles to watchlist
The changes here only apply to the AMC-disabled mode
and only apply when visiting the watchlist with the
following configuraton:
$wgMFUseDesktopSpecialWatchlistPage = [ 'amc' => true, 'base' => true, ];

This change will allow us eventually to remove the special page
in MobileFrontend.

Bug: T109277
Change-Id: I15e95cbae77edde36ca822345a461d791ea5f29d
2024-03-22 14:10:07 -07:00
Ed Sanders 0b7252092e Fix alignment of icons on Special:Preferences
Bug: T360061
Change-Id: Ib13081472222a5e00674a79673223703d055d4ca
2024-03-13 12:13:50 -07:00
Jon Robson aa0a596c7d Define message box, diff colors and missing box shadows for two themes
Bug: T358402
Change-Id: I171b6ad8bb36e52704224c5d28bc6081bc07ace5
2024-03-05 01:51:18 +00:00
Jon Robson 70e68aedd8 Remove the rest of the reset
VISUAL CHANGE:
* Improves top margin on User common.js for code

Bug: T205341
Change-Id: Ia402fd069341da14532d332fab7c109d34c7208d
2024-02-15 19:56:46 +00:00
Jon Robson 145db70286 Reset: Drop p and div inside overlays
Required for proper presentation of page issues overlay
and multimedia viewer.

The div rules were unnecessary.

Bug: T205341
Change-Id: I6fea0f2013a11e5248b71619b795d794c8ed18ad
2024-02-14 22:20:43 +00:00
Steph Toyofuku 91c9e68afb Fix borders in night mode palette
Following a discussion in the ticket, update the places where `#eaecf0`
has been hardcoded to reflect whether it is intended to be a border or
background style and replace with CSS variables where applicable.  Add
`--border-color-muted` as a CSS custom property, and update the night
mode palette to use it.  Lastly, add `--color-error`, to be used in a
subsequent patch

Visual changes, but again gated behind night mode feature flag

Bug: T356825
Change-Id: Icb5741190f3e80a20dcedf9b13d6a34fe619b467
2024-02-14 16:18:21 -05:00
Jon Robson a9f45d2adf Always show patrol link on Minerva skin
Bug: T357213
Change-Id: I36a7ce8a86409087e0142fcc11c6285a327b3875
2024-02-12 21:32:19 +00:00
Jon Robson d5d9fe4ea6 Make thanks button show again
Bug: T357202
Change-Id: Ife53355584da6d7d65413173db8bd93d389996c5
2024-02-12 17:36:33 +00:00
jenkins-bot ec453ac037 Merge "Replace reset with SkinModule normalize" 2024-02-07 14:38:30 +00:00
Jon Robson d9fa5214a7 Reduce font size of diff heading
Bug: T356728
Change-Id: If7c66ca939d9b27d6d8435336514600453e06670
2024-02-06 19:06:52 +00:00
Jon Robson 7c9304cbf4 Replace reset with SkinModule normalize
Retain the existing font,
margin and padding values as without
these we would trigger a visual regression.

Visual change: Four minor visual regression with this change
relating to the button in the AMC CTA Drawer and suggested
edits overlay heading. All are acceptable.

Bug: T205341
Change-Id: I6331d88e5b7301fd13249414350a539738cfae53
2024-02-02 18:00:45 +00:00
Jan Drewniak 59fd0cd5dc Convert all color related Less variables to CSS custom properties
All colors used in Minerva are converted from Less variables
into CSS custom properties. A new file called CSSCustomProperties.less
is created in the skins.minerva.base.styles module to store
these custom properties and an ADR is provided on the rationale for
dropping support for browsers that don't support custom properties.

The new CSS custom properties follow Codex design token conventions
where possible (and noted when not).

Link colors are unique because their styles are defined in core,
so in that case the Less variables values are set to custom properties.
Those values are then fed back into MediaWiki core for core link
styling.

Also adds a temporary night-mode color palette under the
.skin-nightmode-1 class on the <html> element.

Bug: T356074
Change-Id: Ida1f14138f12bd3c600c264bde7b5100f9dbf4ff
2024-02-02 17:57:27 +00:00
Jan Drewniak be0b43042e Convert Less variable names from camelCase to snake-case
Converts all Less variable names from CamelCase to snake-case
per the MediaWiki coding conventions.

Removes the following unused Less variables, mostly
related to icons, since those were converted to Codex:
- @icon-touch-area-sm
- @icon-touch-area-md
- @min-size-icon (replaced with @size-icon-medium)
- @icon-glyph-size-sm
- @icon-size-sm
- @icon-padding-sm
- @icon-glyph-size-md
- @icon-size-md
- @margin-icon-md-labelled

Bug: T350581
Change-Id: I1b16e77942d9bea20dcc5636a63d64aa2325a173
2024-01-31 15:24:31 -08:00
Jon Robson bc2c87f4d0 Fixes: Diff footer is overlaying the text when visual diff is clicked
Bug: T353407
Change-Id: I9246c0df582cab485b425dc77e05dc4435656511
2024-01-31 18:09:04 +00:00
Ed Sanders 8ef144f936 build: Update linters
Change-Id: I5aa3ab891890b52057a5fc855315e5b1b2203037
2024-01-24 22:02:12 +00:00
jenkins-bot c01d353e96 Merge "Long usernames should wrap on history page." 2024-01-19 21:53:37 +00:00
Jon Robson f1c1347d56 Long usernames should wrap on history page.
Bug: T355369
Change-Id: I1f3c3b7e494cc655c98d662e4c0d822da502ea6a
2024-01-18 23:06:15 +00:00
jenkins-bot 6ab98eb62e Merge "Style thanks confirmable panel on mobile" 2024-01-12 17:04:29 +00:00
Jon Robson 5e4e3bd549 Style thanks confirmable panel on mobile
To test you must be using desktop Minerva
This addresses only the problem described in
T353062#9438627.

Bug: T353062
Change-Id: Ia7053b94aae1b6ab48307769de063d825a47c1db
2024-01-12 16:37:21 +00:00