Jump to content

Template talk:Edit summary field

Page contents not supported in other languages.
From Wikipedia, the free encyclopedia

Updating template to OOUI

[edit]

Template-protected edit request on 24 April 2021

[edit]

Hello! Could someone please copy/paste the sandbox back in here? (Somewhat of a pull request for templates.) See the sandbox for a demonstration. This changes the style to the new visual style (VisualEditor & 2017 wikitext editor).

In other words (x to y), please change:

<div class="mw-summary" style="width:50em; margin-left:{{{margin|30px}}}; background-color:#eeeeee; padding:5px;"><!--
-->{{int:summary}}
{{Inputbox|&nbsp;}}
</div>

To:

<div class="mw-summary" style="width:50em; margin-left:{{{margin|30px}}}; background-color:#ffffff; padding:0.5em;"><!--
-->{{int:summary}}<br>
<div class="mw-ui-input" style="width:33em; height:4em;"><span style="color:#72777d;">Describe what you changed</span></div>
</div>

Thank you! Tol | Talk | Contribs 17:02, 24 April 2021 (UTC)[reply]

@Tol: not sure it makes sense to do this - should we be encouraging or assuming VE use among newbies? (I don't think so...) Elli (talk | contribs) 19:13, 26 April 2021 (UTC)[reply]
@Elli: Ah, my mistake. (There are so many editors it gets confusing.) I believe the default is the 2010 WikiEditor? I use the 2017 wikitext editor (the one inside VisualEditor) and mistakenly thought it was the default. Regardless, the box still looks wrong; I'll get it in line with the 2010 editor and fix this request. Tol | Talk | Contribs 19:43, 26 April 2021 (UTC)[reply]
Screenshot of rev 1020034884
Alright; I've updated the sandbox — can you take a look here and let me know what you think? Tol | Talk | Contribs 20:00, 26 April 2021 (UTC)[reply]
 Done Elli (talk | contribs) 20:19, 26 April 2021 (UTC)[reply]
Thanks! Tol | Talk | Contribs 20:43, 26 April 2021 (UTC)[reply]
Broken; reverted. Try again. Mathglot (talk) 18:44, 27 May 2021 (UTC)[reply]
@Mathglot: uh, what was broken exactly? I did test it before implementing and didn't see an issue... Elli (talk | contribs) 22:55, 27 May 2021 (UTC)[reply]
@Elli: This. Mathglot (talk) 00:42, 28 May 2021 (UTC) [reply]
@Mathglot: that's not what it looks like on my end, there's no reason this should be broken. Elli (talk | contribs) 00:45, 28 May 2021 (UTC)[reply]
@Elli: Hm.. that's from mobile; let me get to a computer, and I'll see if it's better there. I'll check a few browsers, as well. Mathglot (talk) 00:48, 28 May 2021 (UTC)[reply]

@Elli:, Ran these tests while not logged in, to make sure it's not something in my common css or js.

Test results of previous (4/26) vs. current (5/26) revision
Tests of two versions of Template
OS browser rev 1020034884 1025468686 view
Win 10 Opera 76.0 broken works
Win 10 Chrome 90.0 broken works
Win 10 Vivaldi 3.5 broken works
iOS Safari 14.6 broken works desktop
iOS Safari 14.6 broken works mobile
iOS Opera 3.0.4 broken works desktop
iOS Opera 3.0.4 broken works mobile
iOS Chrome 90.0 broken works mobile
iOS Chrome 90.0 broken works desktop

Not sure what the next step is; maybe grab the page source of the element and dump it here, so we can look at the html? Can you try some tests, while not logged in? Mathglot (talk) 01:46, 28 May 2021 (UTC)[reply]

Elli, I edited the sandbox as a temporary test, so it contained just the line Before{{Inputbox}}After. What I see there, is an empty input box, flush left, about 3/4 of the width of the page, with the word 'Before' above it, and the word 'After' below it. This test is visible only in rev. 1025527960 of the {{The edit-summary field/sandbox}}. Can you tell me what you see there? Afaict, it is that template that generates the input box that was visible before your version, and that disappeared with your edit.
When I look at the diff of your version of the template, I can see that you removed the line of code that contains template {{Inputbox}}. So, I don't see how it could work on your system without that line. Is it possible that you have an old version of the template cached? What happens if you test your version again, after purging the page? Mathglot (talk) 02:51, 28 May 2021 (UTC)[reply]
@Mathglot: The temporary test in the sandbox shows the same for me as it does for you.
When I try the old version in a private window, it indeed doesn't show the box. However, looking at the code with inspect element shows the exact same thing - <div class="mw-ui-input" style="width:50em; height:2.3em;"><span style="color:#72777d;"></span></div> - in both situations. I guess I might have a custom style-sheet affecting mw-ui-input? (though I don't have an idea why that would be the case) This doesn't appear to be related to caching. Elli (talk | contribs) 03:35, 28 May 2021 (UTC)[reply]
(edit conflict)@Elli: Here's what I see when comparing the generated Html of your version, the version before it, and the sandbox test version in rev. 1025527960:
Comparison of generated Html of sandbox test, and versions 4/26 and 5/26 of the template

Html of sandbox revision 1025527960 contains the following code:

Before{{Inputbox}}After

this generates the following Html on the rendered page:

<div id="mw-content-text" class="mw-body-content mw-content-ltr" lang="en" dir="ltr">
  <div class="mw-parser-output">
    <p>Before</p>
      <div class="messagebox" style="border:0.15em inset; padding:0.1em; margin:0.1em; white-space:nowrap;"> </div>
    <p>After</p>

Andrybak's revision 986882974 of April 26 has the {{Inputbox}} template as well, and generates this:

     <small>(Briefly describe your changes)</small>
  </span>
<div class="messagebox" style="border:0.15em inset; padding:0.1em; margin:0.1em; white-space:nowrap;"> </div>

Elli's revision 1020034884 does not contain template {{Inputbox}}; it generates this:

  (Briefly describe your changes)</small></span><br />
<div class="mw-ui-input" style="width:50em; height:2.3em;">
  <span style="color:#72777d;"></span>
</div>
So, if you're seeing an input box, I don't understand how that's happening, because your version dropped the {{Input box}} template, and seems to be trying to generate the same thing using an Html div with class "mw-ui-input", but a) it doesn't seem to be working, at least, not for everybody; and b) what was the reason for replacing the Inputbox template with direct Html? Mathglot (talk) 03:41, 28 May 2021 (UTC)[reply]
@Mathglot: that indeed seems to be the case. I think the only difference here is if the class "mw-ui-input" properly renders an inputbox.
The reason I did so is described in the original edit request - to match the way the edit summary box actually looks in the modern editor. I didn't write the code - I copied what was in the edit request, tested it (apparently not as thoroughly as it should've been, but I had no reason to expect that it wouldn't work), then implemented it. Elli (talk | contribs) 03:45, 28 May 2021 (UTC)[reply]
@Elli: How do you wish to proceed? Do you want to try just reimplementing it, using the {{Inputbox}} template instead of the mw-ui-input <div>? Mathglot (talk) 03:49, 28 May 2021 (UTC)[reply]
@Mathglot: I think that would kinda defeat the point, which is to not use {{inputbox}} as the way that looks is outdated. If the mw-ui-input <div> doesn't display consistently, for some reason, then the status quo should be kept (though I am curious as to why that is). Elli (talk | contribs) 03:51, 28 May 2021 (UTC)[reply]
@Elli: I forget the details, but I know that certain (probably most) Html elements are turned off on purpose, and anything that interacts with a server, like <input>, are definitely turned off. They've provided some hooks like with {{Inputbox}}, and there's one for searching as well. Here's one that uses {{EditOptions}}:

Edit summary (Briefly describe your changes)

YOUR EDIT SUMMARY goes here,

Empty This is a minor edit Tick Watch this page

By publishing changes, you agree to the Terms of Use, and you irrevocably agree to release your contribution under the CC BY-SA 4.0 License and the GFDL. You agree that a hyperlink or URL is sufficient attribution under the Creative Commons license.

Publish changes Show preview Show changes Cancel
However, if you don't need the input box to be able to have a cursor placed in it, then you can still do it, of course; just create an input box offline looking exactly how you want it, then screenshot the rendered page, and upload it to Wikipedia Commons, and use the image in the template, instead of {{Inputbox}}. That will work for sure. Just please get buy-in from a reasonable number of users of this template; or else, be WP:BOLD and just do it, and see if anybody complains; although that may take a while as it took me a month just to realize it was broken. Mathglot (talk) 04:38, 28 May 2021 (UTC)[reply]
@Mathglot: I see ya. As I don't particularly care about this - pinging Tol who made the original TPER. Elli (talk | contribs) 04:47, 28 May 2021 (UTC)[reply]
(edit conflict) There might be another approach that could work: open a discussion at Template talk:Inputbox, saying that it generates an old-fashioned look, and asking for it to be fixed there. Then all uses of it will improve, not only in this template. Mathglot (talk) 04:48, 28 May 2021 (UTC)[reply]
Huh. I'm busy right now, but I'll take a look at this later. The class is defined as part of the MW UI, but it works for me in every skin. I think the issue is probably somewhere in loading the JS to style the classes. I'll see if I can update Inputbox instead (probably with hard-coded style tags instead of trying to just use the MW UI classes). Thanks for the ping! Tol | Talk | Contribs 05:29, 28 May 2021 (UTC)[reply]

@Elli and Tol: if it matters, you can get the current behavior without the {{Inputbox}} template; just use the messagebox class on any permitted Html tag, such as, say, the <p> tag, along with this style:

  • <p style="border:0.15em inset; padding:0.1em; margin:0.1em; white-space:nowrap;">Hi ,Tol and Elli!</p>

    Hi ,Tol and Elli!

If you can show me what the "newer", or "more modern" output looks like, I'm sure we can imitate it. Mathglot (talk) 03:19, 29 May 2021 (UTC)[reply]

@Elli and Tol: The class isn't even necessary, just the style. Here's a different one that resembles how it looks on my phone; is this what you meant by "modern"?:

Example: fixed typo, added content

Mathglot (talk) 17:44, 1 June 2021 (UTC)[reply]
Not quite.
This is what I see.
It was designed to look like the desktop version of the edit summary in the default WikiEditor (see the Wikimedia OOUI demos; it's similar). Tol | talk | contribs 18:19, 1 June 2021 (UTC)[reply]
@Tol:, thanks for the image. But now I'm more confused than ever, since that image looks very close to the way the template renders it now, at least when I look at it. The only differences I see between the two, is that the template uses inset style on the input text field (i.e., the white box has slightly thicker borders top and left, which gives the illusion that the input field is a bit recessed below the surface of the screen, and the shaded part is grayer (#EEEEEE) compared to your image, which is slightly on the blue side of gray (#EAECF0). Everything else, height, width, border style, font color, size and style, are all equivalent as I see it. If there is anything else you see that is different, could you please describe it?
I've made a change to the sandbox to add a new parameter, |new= and if you include it with any non-empty value (e..g, |new=y), it produces this:

When editing Wikipedia, there is a field labeled "Edit summary" below the main edit box. It looks like this:

Edit summary (Briefly describe your changes)

and if you don't, it produces the default version, namely:

When editing Wikipedia, there is a field labeled "Edit summary" below the main edit box. It looks like this:

Edit summary (Briefly describe your changes)

Does that work for you? Mathglot (talk) 00:53, 6 June 2021 (UTC)[reply]
If that should be the default for all mobile users, we could use {{if mobile}} instead of requiring the |new= param, but allow it as an override. Another way to do it, would be to not change the template at all, but just add code to your commons.css for class mw-summary, and then you can do anything you want. I'll maybe mock up a version of that for you, then we don't have to change the template. Although, that would require opt-in, meaning mobile users wouldn't see it by default, which I gather was not what you were going for. But if you just want it for yourself when logged in, you have that capability already. Mathglot (talk) 01:05, 6 June 2021 (UTC)[reply]
This looks right. I can check the CSS later. The "new" version is the default (WikiEditor) appearance, so it should mimic that appearance as much as possible. Creating a "new inputbox" template might also be a good idea; I can work on that. Tol | talk | contribs 19:06, 6 June 2021 (UTC)[reply]

Try again

[edit]

@Mathglot, Elli: I've fixed the issue (see Template:The edit-summary field/OOUI). I've made three subpages:

This way, they can all be edited and used separately. Because of this, I would like to change this page to use Template:The edit-summary field/OOUI. Do you think that transcluding it or copying it would be the best option? I personally support transcluding it: all past and future templates would have the OOUI version, but people could also transclude another version with the expectation that it won't change in the future. Tol (talk | contribs) @ 02:28, 30 October 2021 (UTC)[reply]

Edit request

[edit]

Due to there being no opposition for a week to the above proposal (I've also included some wording changes), please implement the following change (which can be copied from the sandbox at revision #1054261722):

From To
When editing an article on Wikipedia, you will see a small field labeled "[[Help:Edit summary|Edit summary]]" shown under the main edit box. It looks like this: When editing Wikipedia, there is a field labeled "[[Help:Edit summary|Edit summary]]" below the main edit box. It looks like this:
 
<div class="mw-summary" style="width:50em; {{#if:{{{margin|}}}|margin-left: {{{margin}};|margin: 0 auto;}} background-color:{{#if: {{{new|}}} |#eaecf0 |#eeeeee}}; padding:1em;max-width:100%;box-sizing:border-box"><!-- {{{{{|safesubst:}}}The edit-summary field/{{{1|OOUI}}}}}<!-- Template:The edit-summary field --><noinclude>
-->{{int:summary}}
{{Inputbox|style=border:{{#if: {{{new|}}}| thin solid #ACB2B9; border-radius:2px; | inset }}; width: 100%}}
</div><!-- Template:The edit-summary field --><noinclude>
 
 
{{documentation}} {{documentation}}
<!-- Add categories and interwikis to the /doc subpage, not here! --> <!-- Add categories and interwikis to the /doc subpage, not here! -->
</noinclude> </noinclude>

Thanks, Tol (talk | contribs) @ 01:15, 9 November 2021 (UTC)[reply]

To editor Tol:  done. P.I. Ellsworth - ed. put'r there 04:34, 9 November 2021 (UTC)[reply]
@Paine Ellsworth: Thanks! I always appreciate you taking care of my edit requests. Tol (talk | contribs) @ 04:43, 9 November 2021 (UTC)[reply]
It's my pleasure! Paine  04:48, 9 November 2021 (UTC)[reply]

Requested move 7 December 2022

[edit]
The following is a closed discussion of a requested move. Please do not modify it. Subsequent comments should be made in a new section on the talk page. Editors desiring to contest the closing decision should consider a move review after discussing it on the closer's talk page. No further edits should be made to this discussion.

The result of the move request was: moved. (closed by non-admin page mover)Mdaniels5757 (talk • contribs) 00:52, 15 December 2022 (UTC)[reply]


Template:The edit-summary fieldTemplate:Edit summary field – Per the Manual of Style, Do not use articles (a, an, or the) as the first word. While the MoS does not explicitly apply to non-article-space pages, it establishes standard convention for page titles. Additionally, per Help:Edit summary, "edit summary" is not spelt with a hyphen. Cheers! 🐶 EpicPupper (he/him | talk) 22:44, 7 December 2022 (UTC)[reply]

The discussion above is closed. Please do not modify it. Subsequent comments should be made on the appropriate discussion page. No further edits should be made to this discussion.