Commit graph

1104 commits

Author SHA1 Message Date
Simon Legner 8ecfc1aa7d LightboxImage.originalWidth needs to be a property
mmv.displayPlaceholderThumbnail overrides this value using ImageInfoProvider.

Bug: T64039
Follow-Up: Ic7c5889064492fc88dae33271b635c5327b091e1
Change-Id: I08d024f81336a76439f51b7684446f8725ddb905
2024-06-13 21:48:09 +02:00
Ed Sanders 39968fdb9a Update stylelint-config-wikimedia to 0.17.1
Change-Id: Icaa6d43338a747dbbf1d05d50646d6c7a0edda6d
2024-06-07 17:34:06 +01:00
Ed Sanders 27459e19dc Fix JSDoc complex return types
Change-Id: Ia50a9f086ebb0abcaf0409dac33f66178275cd30
2024-06-07 17:33:16 +01:00
Ed Sanders a74af878a4 Convert jQuery callbacks to arrow functions
Change-Id: I0a9ddd6931ac0f4f30b638491941d624582081ca
2024-06-07 17:33:16 +01:00
Ed Sanders fa18975543 build: Update eslint-config-wikimedia to 0.28.0 and autofix
Change-Id: Iff4b874580768b008427c0bea535ab08cea14a36
2024-06-07 17:33:12 +01:00
Simon Legner 0399ffff40 Api: remove obsolete getQueryField function
Obsolete since Ia4388fe4d5e1d6112a992e826453cd5799a6a4b4.

Bug: T77349
Change-Id: I13efa32786a2ba65edbaedde214057e9efe98145
2024-06-04 19:34:30 +02:00
jenkins-bot c754f877a6 Merge "Add loading bar while loading mmv module" 2024-05-31 20:10:23 +00:00
Derk-Jan Hartman c1a38cfe9e Add loading bar while loading mmv module
When opening the media viewer, we immediately display a black frame,
as we wait for the dependencies etc to load.

Add a codex indeterminate progress bar to this phase as we wait, so
the user has feedback that something is happening.

Bug: T77230
Change-Id: Ibba98150ac2e8034fbfcd586309632704cf0b606
2024-05-31 19:50:19 +00:00
Simon Legner 21438d2b22 Remove FileRepoInfo API and Repo model
Repo was only used in two locations:
1. StripeButtons.set for `isCommons` - can be replaced by `descriptionUrl.includes('//commons.wikimedia.org/')`
2. EmbedFileFormatter.getSiteLink (unused)

Furthermore:
- Simplify StripeButtons (we only have one button)
- Unwrap info objects consisting of `ImageModel` and `Repo`
- Remove unused EmbedFileFormatter.getSiteLink
- Inline EmbedFileFormatter.getCaption and EmbedFileFormatter.getLinkUrl
- Fix JSDoc type `ImageModel`

Reduces mmv bundle size from 28012 to 27246.

Bug: T77349
Change-Id: Ia4388fe4d5e1d6112a992e826453cd5799a6a4b4
2024-05-31 08:13:08 +02:00
Derk-Jan Hartman 82e36ddc61 Use CSS transitions for the progressbar.
Bug: T77349
Change-Id: Ib2e5367b913eaf1f4b8920ed7e78dbdecb01ca1d
2024-05-30 20:00:07 +00:00
jenkins-bot 167978ae5b Merge "Use mw.msg shortcut" 2024-05-29 21:53:16 +00:00
jenkins-bot da7401f983 Merge "Move dialog/reuse dialog code to mmv.ui.reuse module" 2024-05-29 20:23:47 +00:00
Simon Legner a8a5247c5e Use mw.msg shortcut
Change-Id: I5af9cc79d1c6108f643b690ded0dd9b876c3852b
2024-05-29 05:14:01 +00:00
Novem Linguae b02ba00fd0 JSDoc: convert @cfg to @param
Bug: T138401
Change-Id: Ib09d471a28821f54c6f8678e788ba85fbd426d35
2024-05-28 06:25:20 -07:00
Simon Legner 5bee6274e0 Move dialog/reuse dialog code to mmv.ui.reuse module
Initialise the corresponding dialogs upon first use by deferred loading of the mmv.ui.reuse module. Reduces code size of the main mmv module.

Bug: T77349
Change-Id: I672419e1a8af47d192843f7e7a4a038c0d8da2a3
2024-05-28 11:58:44 +02:00
Derk-Jan Hartman f8fe9c0af3 Remove the thumbnail blurring
This removes the thumbnail blurring.

The initial page thumbnail is still in place.
This too can be slightly improved as sizing up to the MMV frame is
done via JS right now, and causes a noticeable change. But we can
simply do this with CSS immediately i think.

Bug: T71609
Bug: T65504
Change-Id: If2a92689be9ee7ecd64561dc224991f31274d58a
2024-05-25 00:56:35 +00:00
Simon Legner 72ba58c134 Merge download/share/embed into mmv.ui.reuse module
Bug: T77349
Change-Id: I19f3444ac3e35d94ba3436e0c1387a1cb5b7e972
2024-05-24 18:27:03 +00:00
Simon Legner 4eaf0725a9 Remove index.js export indirection
Reduces mmv module size from 168.85kB to 167.92kB.

Bug: T77349
Change-Id: I40a4b044ca286dcda948243d5029b3377430d4ff
2024-05-21 22:12:17 +00:00
Simon Legner f00dbbb939 Remove IIFE
Remove Immediately-invoked function expression throughout production code base.

Bug: T77349
Change-Id: Ia31b556975209bbea27df8444b69d924654b181e
2024-05-21 22:31:15 +02:00
jenkins-bot d4cded1d88 Merge "Simplify MetadataPanel.setLocationData" 2024-05-21 19:27:04 +00:00
jenkins-bot bc3cc059bb Merge "Revert "Revert "Add position handling to URLs""" 2024-05-21 19:27:02 +00:00
TheDJ 24ee9dc48d Revert "Revert "Add position handling to URLs""
This reverts commit 5b89420182.

Reason for revert: restore previously reverted patch

Bug: T64039
Change-Id: Iac4486ae785f308d2a572f383073cfb73c7cb523
2024-05-21 19:09:16 +00:00
jenkins-bot 45ea6b65ab Merge "Touch events should make the controls visible" 2024-05-20 23:26:38 +00:00
Derk-Jan Hartman f592eeea3b Remove unused code from mmv.ui
Bug: T77349
Change-Id: I9e26dc68859d9dca3c6066837253e85a19039ce0
2024-05-20 23:02:04 +00:00
TheDJ 5b89420182 Revert "Add position handling to URLs"
This reverts commit de56d4a6de.

Reason for revert: breaks resize handler. Lets revert before branch cut.

Change-Id: I1e4d57d8384ab434f17968100f8bea1bf90c40ff
2024-05-20 20:51:59 +00:00
Derk-Jan Hartman 8940fbf6a7 Touch events should make the controls visible
Bug: T364942
Change-Id: If423570ba6e7b99f7383e71108fa2048852616b2
2024-05-20 22:04:59 +02:00
Simon Legner de56d4a6de Add position handling to URLs
- Instantiate LightboxImage object in mmv.bootstrap
- Add LightboxImage.position field: relative position of this image to others with same file
- Add optional routing suffix /:position

Bug: T64039
Change-Id: Ic7c5889064492fc88dae33271b635c5327b091e1
2024-05-20 18:36:23 +00:00
Simon Legner dad874d6ce Simplify MetadataPanel.setLocationData
Extract common function convertDegMinSec for latitude and longitude.

Change-Id: I7037ab1d1444b670f37834f178220fc7cabd433f
2024-05-18 17:27:20 +00:00
jenkins-bot 78f88fd300 Merge "Extract mmv.ui.restriction module" 2024-05-17 21:08:54 +00:00
Simon Legner c53047d181 Extract mmv.ui.restriction module
This module contains the restriction icons. The main mmv module shrinks from 196.88kB to 174.97kB.

Bug: T77349
Change-Id: I78d16b487f5272a8f779f3484c883d113db1878b
2024-05-17 20:51:05 +00:00
jenkins-bot ae063e56f3 Merge "Use heavier box shadow for the dialog" 2024-05-17 15:45:36 +00:00
jenkins-bot 1246e5bd5f Merge "Allow striken text in MMV descriptions" 2024-05-17 13:20:07 +00:00
jenkins-bot 0e2b527950 Merge "Use the Translated namespace in the metadata panel" 2024-05-16 23:34:10 +00:00
jenkins-bot 9ac1994cf8 Merge "Make functions in Utils class static" 2024-05-16 23:30:30 +00:00
jenkins-bot c33efb3919 Merge "Revert "Fix Codex icon on progressive buttons in dark mode"" 2024-05-16 23:19:33 +00:00
VolkerE 7164a90d48 Revert "Fix Codex icon on progressive buttons in dark mode"
This reverts commit 2f05b66988.

Reason for revert: Fixed upstream in v1.6.0

Bug: T364254
Change-Id: I0d0923254f7bdb53d03f60b3ff8a997605f7a66b
2024-05-16 23:16:28 +00:00
Simon Legner cfb250a57e Make functions in Utils class static
Bug: T337102
Change-Id: Ia186ba355d7da6034309453788f2f84d6b8de899
2024-05-16 21:52:14 +02:00
Derk-Jan Hartman 018f21ba46 Use heavier box shadow for the dialog
White on white got a bit confusing. Use a much heavier
box-shadow to give better distinction for the elements.

Alternative is to use a border of the subtle color, but doesn't look
as nice to me.

Bug: T365192
Change-Id: I0bf760f0987b2c04a66b00f24398561588bd719a
2024-05-16 21:28:10 +02:00
Derk-Jan Hartman 4b45cd438f Use the Translated namespace in the metadata panel
Bug: T365124
Change-Id: Ibf932f266a66ce34909290a6441a87caeec05431
2024-05-16 19:02:25 +00:00
Derk-Jan Hartman b7a5572524 Allow striken text in MMV descriptions
Bug: T288492
Change-Id: I3b7c2998d72cf12b7420aebe02e6e749b9f5f9da
2024-05-16 00:47:32 +02:00
jenkins-bot 11efc235c3 Merge "Use jQuery.data in favour of jQuery.attr" 2024-05-15 22:11:46 +00:00
jenkins-bot e8b55d8204 Merge "Remove some unreachable/no longer needed code" 2024-05-15 22:11:05 +00:00
Simon Legner e50a8b604a Remove extraStatsDeferred
Seems to be a leftover from performance instrumentation I97d41be93849b2ae9d1adba6660546ea716657fd.

Change-Id: Iaba7d23fd526db772528d3ccb74889e7000b3d71
2024-05-15 20:59:58 +02:00
Derk-Jan Hartman de81163226 Remove some unreachable/no longer needed code
These browser fallbacks etc are no longer needed
We even had an unused file

Change-Id: I278f8adbb72cbd587b4a812aa11f40a63d334c8c
2024-05-15 20:48:16 +02:00
Simon Legner 628979d317 Use jQuery.data in favour of jQuery.attr
Ref: https://api.jquery.com/data/

Change-Id: I4ce901a8b030c4864814b8bed14dc0049fd788d7
2024-05-15 20:28:58 +02:00
jenkins-bot 870d9d1d0b Merge "Make functions in HtmlUtils class static" 2024-05-12 14:43:02 +00:00
jenkins-bot c9bb15f15d Merge "Use Title.getPrefixedText for localized File: namespace" 2024-05-12 14:43:00 +00:00
jenkins-bot cc0b57939a Merge "Restore success notification for copy-to-clipboard buttons" 2024-05-12 14:42:57 +00:00
Simon Legner 99275d517a Make functions in HtmlUtils class static
Bug: T337102
Change-Id: I665e6e965d8b595ef9b1b3a58164d272007c3f81
2024-05-10 18:42:24 +03:00
Simon Legner e0d3db89cd Fix dialog background color
Use background-color-base instead of background-color-neutral.

Bug: T363776
Change-Id: Id954acdc07a33ae7bb1a0f57367b87b1d867bb0c
2024-05-09 21:47:20 +03:00