Task description update

• ADDED the following language to the "Problem" section, inspired by the feedback @Tacsipacsi shared in T267595#6663423: "...and as distinct from the content that already exists on the page."

@Tacsipacsi, can you please give the task description a read and let us know if the issue you raised in T267595#6663423 is accurately and exhaustively represented in the "Problem" section?

In T269157#6665316, @ppelberg wrote:

@Tacsipacsi, can you please give the task description a read and let us know if the issue you raised in T267595#6663423 is accurately and exhaustively represented in the "Problem" section?

Mostly yes. I couldn’t write it any better.

In Description, @ppelberg wrote:

Proposed Solution: Add a background color

This looks risky to me—pages can have all sorts of background colors coming from different sources, including colors set in wikicode (within that page or via some template), on-wiki CSS (MediaWiki:Common.css, MediaWiki:Vector.css, user common.css, user vector.css, user global.css, gadgets etc.), skins’ stock CSS (see e.g. Cologne Blue), or even set on client side using JavaScript (e.g. MediaWiki:Gadget-userpage-customization.js on huwiki). These (and probably more) techniques can set any color (or even a background image), and no matter what color DT uses, it will look awful together with some of these or become simply indistinguishable from them. I’d like to rather see something around spacing and/or a border/separating line that uses standard text color.

This looks risky to me—pages can have all sorts of background colors coming from different sources, including colors set in wikicode (within that page or via some template), on-wiki CSS (MediaWiki:Common.css, MediaWiki:Vector.css, user common.css, user vector.css, user global.css, gadgets etc.), skins’ stock CSS (see e.g. Cologne Blue), or even set on client side using JavaScript (e.g. MediaWiki:Gadget-userpage-customization.js on huwiki). These (and probably more) techniques can set any color (or even a background image), and no matter what color DT uses, it will look awful together with some of these or become simply indistinguishable from them. I’d like to rather see something around spacing and/or a border/separating line that uses standard text color.

Yes agreed. I think that we should go an approach that lightly outlines the component as a unit, and to balance out the spacing. I made a few mockups that show what this will look like on a user talk page (but of course this will work in other Talk environments):

Source Mode:

Visual Mode:

Action(s)

• @ppelberg to add final mocks @iamjessklein produced in T269157#6708721 to the task description so Engineering can implement this.

In T269157#6710616, @ppelberg wrote:

Action(s)

• @ppelberg to add final mocks @iamjessklein produced in T269157#6708721 to the task description so Engineering can implement this.

✅done; see task description.

In T269157#6708721, @iamjessklein wrote:

I made a few mockups that show what this will look like on a user talk page (but of course this will work in other Talk environments):

Could you please make some mockups that include edit notices? That is what’s most confusing right now (it just seems to be part of the previous section). Otherwise, it looks good to me.

In T269157#6711171, @Tacsipacsi wrote:

In T269157#6708721, @iamjessklein wrote:

I made a few mockups that show what this will look like on a user talk page (but of course this will work in other Talk environments):

Could you please make some mockups that include edit notices? That is what’s most confusing right now (it just seems to be part of the previous section). Otherwise, it looks good to me.

We appreciate you having a look at these mockups, @Tacsipacsi.

Regarding the edit notice component, we're going to implement this after the tool is initially made available as a beta feature. This work on the "edit notice component" will happen in T269033.

Change 655990 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/DiscussionTools@master] Padding and border around new topic tool

https://gerrit.wikimedia.org/r/655990

Test wiki created on Patch Demo by ESanders (WMF) using patch(es) linked to this task:

http://patchdemo.wmflabs.org/wikis/9da5582594824af2474a3c9f52cbf99a/w/

Nice work @Esanders +1 to merge

cc @ppelberg

Change 655990 merged by jenkins-bot:
[mediawiki/extensions/DiscussionTools@master] Padding and border around new topic tool

https://gerrit.wikimedia.org/r/655990