Maniphest T196375

No right margin is needed for the last child of the save dialog's checkboxes block
Closed, ResolvedPublic1 Estimated Story Points
Actions

Assigned To

Authored By

	Jack_who_built_the_house
	Jun 4 2018, 3:02 PM

Description

In the save dialog's checkboxes block, .oo-ui-fieldLayouts have margin-right equal to 1.5em. But the last child doesn't need it since the checkboxes block has padding already:

After the change, the checkboxes in ruwiki won't be split into two lines anymore:

Existing code:

.ve-ui-mwSaveDialog-checkboxes .oo-ui-fieldLayout {
    display: inline-block;
    margin: 0 1.5em 0 0;
}

Proposed addition:

.ve-ui-mwSaveDialog-checkboxes .oo-ui-fieldLayout:last-child {
    margin-right: 0;
}

(Also, if the checkboxes are split into two lines, the bottom margin is bigger than the top, but I guess a separate task is needed for this.)

Details

	Subject	Repo	Branch	Lines +/-
	ve.ui.MWSaveDialog: Remove unnecessary margin after last checkbox	mediawiki/extensions/VisualEditor	master	+4 -0

Customize query in gerrit

Event Timeline

Jack_who_built_the_house created this task.Jun 4 2018, 3:02 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 4 2018, 3:02 PM

Jack_who_built_the_house renamed this task from No right margin is needed for the last child in the save dialog's checkboxes block to No right margin is needed for the last child of the save dialog's checkboxes block.Jun 4 2018, 3:04 PM

stjn subscribed.Jun 4 2018, 3:11 PM

Sounds reasonable. Do you want to submit a patch? You can find this code in the mediawiki/extensions/VisualEditor Git repository, in
modules/ve-mw/ui/styles/dialogs/ve.ui.MWSaveDialog.css.

That space is reserved for the "remaining characters" counter. We recently changed it to only show when there are <100(?) chars remaining, hence the confusion:

We could change it to only add the margin when the counter is show (which is very rarely), but also having the dialog reflow would also be sub-optimal (you may enough need to recalculate the whole height of the modal).

This margin is unnecessary though, and I think that's what we're proposing to remove:

Ok, I was going off the margin shown in the first screenshot in the description.

Change 438131 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] ve.ui.MWSaveDialog: Remove unnecessary margin after last checkbox

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

gerritbot added a project: Patch-For-Review.Jun 8 2018, 1:27 AM

Change 438131 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] ve.ui.MWSaveDialog: Remove unnecessary margin after last checkbox

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

matmarex claimed this task.Jun 8 2018, 10:26 AM

matmarex edited projects, added VisualEditor (Current work), VisualEditor-MediaWiki; removed Patch-For-Review, VisualEditor.

matmarex moved this task from Incoming to Product owner review on the VisualEditor (Current work) board.

ReleaseTaggerBot added a project: MW-1.32-notes (WMF-deploy-2018-06-12 (1.32.0-wmf.8)).Jun 8 2018, 11:00 AM

Thank you, @matmarex. Sorry, had to time.

• Deskana closed this task as Resolved.Jun 11 2018, 3:05 PM

• Deskana triaged this task as Medium priority.

Jdforrester-WMF set the point value for this task to 1.Jun 13 2018, 6:26 PM

Ryasmeen added a project: Verified.Jun 14 2018, 12:06 AM

• Vvjjkkii renamed this task from No right margin is needed for the last child of the save dialog's checkboxes block to gobaaaaaaa.Jul 1 2018, 1:05 AM

• Vvjjkkii reopened this task as Open.

• Vvjjkkii removed matmarex as the assignee of this task.

• Vvjjkkii raised the priority of this task from Medium to High.

• Vvjjkkii added projects: CheckUser, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), Tamil-Sites, Gamepress, Hashtags, Jade, KartoEditor, Language-2018-Apr-June, New-Editor-Experiences, Mail, TCB-Team (now WMDE-TechWish).

• Vvjjkkii updated the task description. (Show Details)

• Vvjjkkii removed the point value for this task.

• Vvjjkkii removed subscribers: gerritbot, Aklapper.

CommunityTechBot renamed this task from gobaaaaaaa to No right margin is needed for the last child of the save dialog's checkboxes block.Jul 2 2018, 4:43 AM

CommunityTechBot closed this task as Resolved.

CommunityTechBot assigned this task to matmarex.

CommunityTechBot lowered the priority of this task from High to Medium.

CommunityTechBot set the point value for this task to 1.

CommunityTechBot updated the task description. (Show Details)

CommunityTechBot removed projects: TCB-Team (now WMDE-TechWish), Mail, New-Editor-Experiences, Language-2018-Apr-June, KartoEditor, Jade, Hashtags, Gamepress, Tamil-Sites, Connected-Open-Heritage-Batch-uploads (RAÄ-KMB_1_2017-02), CheckUser.

CommunityTechBot added subscribers: gerritbot, Aklapper.

	F18747839: image.png
	Jun 4 2018, 5:08 PM

	F18747202: image.png
	Jun 4 2018, 4:55 PM

No right margin is needed for the last child of the save dialog's checkboxes blockClosed, ResolvedPublic1 Estimated Story PointsActions

Description

Details

Event Timeline

No right margin is needed for the last child of the save dialog's checkboxes block
Closed, ResolvedPublic1 Estimated Story Points
Actions