Switching modes recreates the toolbar, so if someone has already edited the
save summary it shouldn't update the initialEditSummary to that.
Bug: T159686
Change-Id: I4109442853916bebae2bce804fcd6777e3eb6d34
The non-FullRestbaseURL branch didn't handle converting the provided wikitext,
causing data loss when switching from source to visual modes.
Bug: T158692
Change-Id: I283afc4be1e322228d6ec572bfa8542ba8fd9ce7
It's actually a bit surprising that the only thing broken by using
OptionWidget instead of MenuOptionWidget is some missing styles.
Change-Id: I3961fedbfc61f2f17d91dc7375d47a3cdff2a257
Ensure dm.LanguageAnnotation exists when it runs, otherwise
it won't match and the context will fall back to a tool
context which doesn't have the clear annotation button.
Change-Id: Ie5474a93fcc1beab2a752bbc3beb17b3ef9cab76
Switching from visual to source reruns setupUnloadHandlers, which caused an
infinite loop where it thought it was its own fallback pre-existing handler.
Bug: T153346
Change-Id: I8df55a5395ede02fc34ec47a94f2c780dc08595f
New changes:
9dbb3729b DiffElement: Measure box-shadow borders in pixels to avoid blurring
cbcaed302 ve.ce.Surface: Prevent horizontal scrollbars when hovering FocusableNodes in RTL
b715b1ad6 DiffElement: Use a real ve.Range instead of a plain object
d2567df6c DesktopContext: Unhide context before positioning popup
1afcce366 Localisation updates from https://translatewiki.net.
c31dccebd Update OOjs UI to v0.19.5
Bug: T159472
Change-Id: Icbff2ef3bdb3269db53183f27e2b682a755ee781
It doesn't work, you can't turn a MWMagicLinkNode into text by
clearing its styling. (You can still do that by clicking 'Edit',
'Convert to simple link', 'Clear styling'.)
`clearable = true` was being inherited from ve.ui.LinkContextItem.
Change-Id: Ib40e952554966fb74b3e72662a6e1e44ad748a57
Since December, the handler now expects to be passed the mode we're switching
to. Since we're filtering for just the edit link and not edit source, we can
hardcode 'visual' here.
Bug: T159374
Change-Id: I160045dd59ff1b4c2d180dbbe37280a1839886df
Though neat, this hack to get the internal list doesn't seem to be used
and creates an invalid document (just an internal list with no content,
so the internal list is at the start and the default selection position
is inside it). Instead, use the ve.dm.Document.static.newBlankDocument()
utility function that was made for this use case.
Bug: T153509
Change-Id: Iba5931cb5e3f939a803c1d59fa0d8f30a0513f84
New changes:
f83a68c22 ve.ce.ContentBranchNode: Don't set 'white-space: normal' on `<pre>` tags
ee5920781 DiffElement: Fix remaining class->data-attribute conversions
a07c9e345 DiffElement: De-clutter left margin, removing the blue border
3056a4a46 ve.dm.Document: Provide a utility method to provide a blank Document
Bug: T159231
Change-Id: I3b71d995c915fa6ea10a0eea7cb727ee68733ae9
* Ensure DAT styles are fully scoped to DAT instances
* Ensure that generic desktop target rules are in the correct file
* Remove old unused rules
Change-Id: I466a0547ff9a0fa1102aac0b1e501a6ba6863ca3
New changes:
7da907be0 Highlight comment changes in visual diff and fix whitespace
77accf6f6 Have toolbar append toolbarDialogs container
6985593cd Localisation updates from https://translatewiki.net.
9140a5991 DiffElement: Avoid modifying class attributes on DM HTML
cdcbbdde3 Remove used 'header' params from toolbar config
15f36bc75 Update OOjs UI to v0.19.4
Bug: T149821
Bug: T158895
Change-Id: I5bfc595b6b43173e76fc8cd45e47fb21d5ccaea7
Pencil is the icon we use for 'edit' elsewhere in the UI.
Use the eye icon for the VE tool to specify 'visual'.
Bug: T116417
Change-Id: I12b6bab2a52758685abde04579b274a32d651174
