Commit graph

356 commits

Author SHA1 Message Date
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
Jon Robson 72726eac36 Fix text overflow in history page
This omitted the important overflow and text-overflow
rules that we use in MobileFrontend. The line height also
needs a slight adjustment.

Bug: T354218
Change-Id: I4b8dee3395603933aba2818725b1f8fbbde0a2a6
2024-01-11 19:38:15 +00:00
jenkins-bot 459d04d121 Merge "History: Make sure entire history row is clickable" 2023-12-20 19:13:06 +00:00
Jon Robson 67403c89b6 Switch to non-compact mode on tablet breakpoint instead of desktop
Bug: T353478
Change-Id: I2b6141f949c663f3d3d720f087d0d0a0936f89f3
2023-12-20 18:42:31 +00:00
Jon Robson 8d305924c9 History: Make sure entire history row is clickable
Bug: T353485
Change-Id: Ie10088636e727c2d5b263426073a02a8f978b692
2023-12-20 18:26:12 +00:00
Jon Robson b1b374b5b3 Relax CSS selector to apply to checkboxes as well as radio buttons
Bug: T353401
Change-Id: I520261fabf404c3a4f96ae4449ce9c445e7da270
2023-12-13 14:00:46 -10:00
Ed Sanders 86289e7b26 Show diff switcher on narrow screens
The visual diff is sometimes cluttered at narrow widths, but as
we show the switcher on tablet and desktop, and it stores a preference,
users may see the visual diff on phones anyway.

The layout is sometimes cluttered but not completely useless, and
users can always switch back.

As a follow-up we should improve the layout of the visual diff sidebar
on narrow screens.

Bug: T351946
Change-Id: I5533126195331b46040e708b7b35627e74a3001d
2023-11-30 18:43:39 +00:00