Temporary Disabled. :) please Go back ⚓ T95989 FlaggedRevs edit notices don't render properly in VisualEditor www.fgks.org » Address: [go: up one dir, main page] Include Form Remove Scripts Accept Cookies Show Images Show Referer Rotate13 Base64 Strip Meta Strip Title Session Cookies Page MenuHomePhabricatorSearchConfigure Global SearchLog InCreate Task Maniphest T95989 FlaggedRevs edit notices don't render properly in VisualEditorClosed, ResolvedPublic1 Estimated Story PointsActionsEdit TaskEdit Related Tasks...Create SubtaskEdit Parent TasksEdit SubtasksMerge Duplicates InClose As DuplicateEdit Related Objects...Edit CommitsEdit MocksSubscribeMute NotificationsProtect as security issueAward TokenFlag For LaterAssigned ToKrinkleAuthored BySunpriatApr 14 2015, 3:25 AM2015-04-14 03:25:47 (UTC+0)TagsVisualEditor (FY 18-19 Q3/Q4)VisualEditor-MediaWiki (Backlog)OOUI (Reviewing)VisualEditor 2014/15 Q4 blockers (Done)WMF-deploy-2015-04-15_(1.26wmf2) (Backlog)WMF-deploy-2015-04-08_(1.26wmf1) (Backlog)WMF-deploy-2015-04-01_(1.25wmf24) (Backlog)User-notice-archive (Backlog)Referenced FilesF112738: Снимок33.PNGApr 14 2015, 11:51 AM2015-04-14 11:51:02 (UTC+0)F112723: Screen_Shot_2015-04-14_at_12.17.13.pngApr 14 2015, 11:18 AM2015-04-14 11:18:49 (UTC+0)F112710: Снимок31.PNGApr 14 2015, 10:37 AM2015-04-14 10:37:26 (UTC+0)F112711: Снимок32.PNGApr 14 2015, 10:37 AM2015-04-14 10:37:26 (UTC+0)SubscribersAklappergerritbotKrinklematmarexSunpriatDescriptionI try to edit any article in VE in https://ru.wikipedia.org/ and a pop-up window "edit notices" is empty, but there is a text "1 2 3...n notices"DetailsSubjectRepoBranchLines +/-MWPopupTool: Don't assume editnotice html has a single wrapping elementmediawiki/extensions/VisualEditorwmf/1.25wmf24+6 -6MWPopupTool: Don't assume editnotice html has a single wrapping elementmediawiki/extensions/VisualEditorwmf/1.26wmf1+6 -6MWPopupTool: Don't assume editnotice html has a single wrapping elementmediawiki/extensions/VisualEditormaster+6 -6Customize query in gerritRelated ObjectsMentionsMentioned In T95822: Assist de/it/pl/pt/ru-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice"T97644: VE: empty "edit notices"rEVED668a7c1aace5: MWPopupTool: Don't assume editnotice html has a single wrapping elementrEVEDd7f851ba063a: MWPopupTool: Don't assume editnotice html has a single wrapping elementrEVED66c9d538c504: MWPopupTool: Don't assume editnotice html has a single wrapping element Mentioned Here T45013: VisualEditor: Deal with en.wikipedia.org's edit notice systemT95822: Assist de/it/pl/pt/ru-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice"T91715: Implement (part of) the English Wikipedia's edit notice system in MediaWiki core and transition the wikis to use it Event TimelineSunpriat created this task.Apr 14 2015, 3:25 AM2015-04-14 03:25:47 (UTC+0)Sunpriat raised the priority of this task from to Needs Triage.Sunpriat updated the task description. (Show Details)Sunpriat added a project: VisualEditor.Sunpriat subscribed.Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 14 2015, 3:25 AM2015-04-14 03:25:47 (UTC+0)Aklapper added a comment.Apr 14 2015, 10:23 AM2015-04-14 10:23:26 (UTC+0)Comment ActionsThanks for taking the time to report this! Could you please provide a specific example link to reproduce, and the steps how to trigger such a popup window?Sunpriat added a comment.Edited · Apr 14 2015, 10:37 AM2015-04-14 10:37:26 (UTC+0)Comment Actionsany browser, any article, any wiki https://en.wikipedia.org/wiki/Mulan_II?veaction=edit https://ru.wikipedia.org/wiki/%D0%9C%D1%83%D0%BB%D0%B0%D0%BD_2?veaction=editKrinkle subscribed.Edited · Apr 14 2015, 11:18 AM2015-04-14 11:18:49 (UTC+0)Comment ActionsNot all pages are affected. I can reproduce the issue on those two pages, but the majority of other pages are unaffected. These pages have in common that they're protected with FlaggedRevs. Compare to the wikitext editor: {"visualeditor":{"result":"success","notices":{"revreview-locked": <b>Note:</b> Edits to this page are subject to review (<a href="/wiki/Wikipedia:Pending_changes" title="Wikipedia:Pending changes">help</a>). <div id="mw-fr-logexcerpt"><ul> .. </ul> </div> Only the first node (the bold word) shows up in VisualEditor. Krinkle renamed this task from VE: in ru.wiki does not show the edit notices to FlaggedRevs edit notices don't render properly in VisualEditor.Apr 14 2015, 11:23 AM2015-04-14 11:23:35 (UTC+0)Krinkle claimed this task.Krinkle added a project: VisualEditor-MediaWiki.Krinkle set Security to None.gerritbot subscribed.Apr 14 2015, 11:25 AM2015-04-14 11:25:35 (UTC+0)Comment ActionsChange 204038 had a related patch set uploaded (by Krinkle): MWPopupTool: Don't assume editnotice html has a single wrapping element https://gerrit.wikimedia.org/r/204038gerritbot added a project: Patch-For-Review.Apr 14 2015, 11:25 AM2015-04-14 11:25:35 (UTC+0)Sunpriat added a comment.Edited · Apr 14 2015, 11:51 AM2015-04-14 11:51:02 (UTC+0)Comment Actionsrandom page: https://ru.wikipedia.org/wiki/%D0%9A%D1%80%D0%BD%D0%B0%D1%87?veaction=edit no FlaggedRevs protectKrinkle added a comment.Apr 14 2015, 12:21 PM2015-04-14 12:21:17 (UTC+0)Comment Actions In T95989#1206273, @Sunpriat wrote: random page: https://ru.wikipedia.org/wiki/%D0%9A%D1%80%D0%BD%D0%B0%D1%87?veaction=edit no FlaggedRevs protect That is a separate issue. The problem there is that ru.wikipedia.org has a global edit notice at MediaWiki:Editnotice-0 but is invisible to most users (only visible to sysops). See also T91715. I've converted it to use MediaWiki:Editnotice-notext. Fixed!matmarex subscribed.Apr 14 2015, 3:05 PM2015-04-14 15:05:22 (UTC+0)Comment ActionsSame issue as T95822?gerritbot added a comment.Apr 14 2015, 3:15 PM2015-04-14 15:15:07 (UTC+0)Comment ActionsChange 204038 merged by jenkins-bot: MWPopupTool: Don't assume editnotice html has a single wrapping element https://gerrit.wikimedia.org/r/204038Krinkle mentioned this in rEVED66c9d538c504: MWPopupTool: Don't assume editnotice html has a single wrapping element.Apr 14 2015, 3:15 PM2015-04-14 15:15:33 (UTC+0)Krinkle added a comment.Apr 14 2015, 3:17 PM2015-04-14 15:17:35 (UTC+0)Comment Actions In T95989#1206646, @matmarex wrote: Same issue as T95822? Since those don't show anything at all, it's likely not the FlaggedRevs issue but the Editnotice-notext issue. Which is discussed at T91715 and T45013. Though doesn't have a proper task yet. Let's use T95822 for that.Jdforrester-WMF closed this task as Resolved.Apr 14 2015, 3:41 PM2015-04-14 15:41:09 (UTC+0)Jdforrester-WMF triaged this task as Medium priority.Jdforrester-WMF edited projects, added OOUI, OOjs-UI-next-release; removed Patch-For-Review.Jdforrester-WMF added a project: VisualEditor 2014/15 Q4 blockers.Jdforrester-WMF edited a custom field.Jdforrester-WMF raised the priority of this task from Medium to High.Apr 14 2015, 5:53 PM2015-04-14 17:53:21 (UTC+0)Jdforrester-WMF moved this task from Nominated to Done on the VisualEditor 2014/15 Q4 blockers board.gerritbot added a comment.Apr 14 2015, 10:14 PM2015-04-14 22:14:22 (UTC+0)Comment ActionsChange 204167 had a related patch set uploaded (by Catrope): MWPopupTool: Don't assume editnotice html has a single wrapping element https://gerrit.wikimedia.org/r/204167gerritbot added a project: Patch-For-Review.Apr 14 2015, 10:14 PM2015-04-14 22:14:23 (UTC+0)Comment ActionsChange 204168 had a related patch set uploaded (by Catrope): MWPopupTool: Don't assume editnotice html has a single wrapping element https://gerrit.wikimedia.org/r/204168gerritbot added a comment.Apr 14 2015, 10:52 PM2015-04-14 22:52:42 (UTC+0)Comment ActionsChange 204167 merged by jenkins-bot: MWPopupTool: Don't assume editnotice html has a single wrapping element https://gerrit.wikimedia.org/r/204167Catrope mentioned this in rEVEDd7f851ba063a: MWPopupTool: Don't assume editnotice html has a single wrapping element.Apr 14 2015, 10:53 PM2015-04-14 22:53:04 (UTC+0)gerritbot added a comment.Apr 14 2015, 11:01 PM2015-04-14 23:01:25 (UTC+0)Comment ActionsChange 204168 merged by jenkins-bot: MWPopupTool: Don't assume editnotice html has a single wrapping element https://gerrit.wikimedia.org/r/204168Catrope mentioned this in rEVED668a7c1aace5: MWPopupTool: Don't assume editnotice html has a single wrapping element.Apr 14 2015, 11:01 PM2015-04-14 23:01:35 (UTC+0)Jdforrester-WMF edited projects, added WMF-deploy-2015-04-15_(1.26wmf2), WMF-deploy-2015-04-08_(1.26wmf1), WMF-deploy-2015-04-01_(1.25wmf24); removed Patch-For-Review, OOjs-UI-next-release.Apr 14 2015, 11:47 PM2015-04-14 23:47:20 (UTC+0)matej_suchanek added a project: User-notice.Apr 17 2015, 12:43 PM2015-04-17 12:43:45 (UTC+0)matej_suchanek moved this task from To Triage to In current Tech/News draft on the User-notice board.nshahquinn-wmf moved this task from To Triage to FY 18-19 Q3/Q4 on the VisualEditor board.Apr 17 2015, 7:39 PM2015-04-17 19:39:39 (UTC+0)• gpaumier moved this task from In current Tech/News draft to Recently announced in Tech/News on the User-notice board.Apr 20 2015, 4:11 PM2015-04-20 16:11:48 (UTC+0)• gpaumier moved this task from Recently announced in Tech/News to Already announced/Archive on the User-notice board.Apr 23 2015, 9:47 PM2015-04-23 21:47:17 (UTC+0)Jdforrester-WMF moved this task from Backlog to Reviewing on the OOUI board.Apr 29 2015, 9:35 PM2015-04-29 21:35:50 (UTC+0)Sunpriat mentioned this in T97644: VE: empty "edit notices".Apr 30 2015, 1:49 PM2015-04-30 13:49:10 (UTC+0)Sunpriat mentioned this in T95822: Assist de/it/pl/pt/ru-wiki communities in updating their edit-notice systems to stop hidden content showing as "1 notice".May 1 2015, 6:41 AM2015-05-01 06:41:20 (UTC+0)Ladsgroup edited projects, added User-notice-archive; removed User-notice.Aug 13 2022, 1:53 PM2022-08-13 13:53:52 (UTC+0) · Log In to Comment
I try to edit any article in VE in https://ru.wikipedia.org/ and a pop-up window "edit notices" is empty, but there is a text "1 2 3...n notices"
Thanks for taking the time to report this! Could you please provide a specific example link to reproduce, and the steps how to trigger such a popup window?
any browser, any article, any wiki
Not all pages are affected. I can reproduce the issue on those two pages, but the majority of other pages are unaffected.
These pages have in common that they're protected with FlaggedRevs. Compare to the wikitext editor:
{"visualeditor":{"result":"success","notices":{"revreview-locked": <b>Note:</b> Edits to this page are subject to review (<a href="/wiki/Wikipedia:Pending_changes" title="Wikipedia:Pending changes">help</a>). <div id="mw-fr-logexcerpt"><ul> .. </ul> </div>
Only the first node (the bold word) shows up in VisualEditor.
Change 204038 had a related patch set uploaded (by Krinkle): MWPopupTool: Don't assume editnotice html has a single wrapping element
https://gerrit.wikimedia.org/r/204038
random page: https://ru.wikipedia.org/wiki/%D0%9A%D1%80%D0%BD%D0%B0%D1%87?veaction=edit
In T95989#1206273, @Sunpriat wrote: random page: https://ru.wikipedia.org/wiki/%D0%9A%D1%80%D0%BD%D0%B0%D1%87?veaction=edit no FlaggedRevs protect
That is a separate issue. The problem there is that ru.wikipedia.org has a global edit notice at MediaWiki:Editnotice-0 but is invisible to most users (only visible to sysops). See also T91715. I've converted it to use MediaWiki:Editnotice-notext. Fixed!
Same issue as T95822?
Change 204038 merged by jenkins-bot: MWPopupTool: Don't assume editnotice html has a single wrapping element
In T95989#1206646, @matmarex wrote: Same issue as T95822?
Since those don't show anything at all, it's likely not the FlaggedRevs issue but the Editnotice-notext issue. Which is discussed at T91715 and T45013. Though doesn't have a proper task yet. Let's use T95822 for that.
Change 204167 had a related patch set uploaded (by Catrope): MWPopupTool: Don't assume editnotice html has a single wrapping element
https://gerrit.wikimedia.org/r/204167
Change 204168 had a related patch set uploaded (by Catrope): MWPopupTool: Don't assume editnotice html has a single wrapping element
https://gerrit.wikimedia.org/r/204168
Change 204167 merged by jenkins-bot: MWPopupTool: Don't assume editnotice html has a single wrapping element
Change 204168 merged by jenkins-bot: MWPopupTool: Don't assume editnotice html has a single wrapping element