Page MenuHomePhabricator

Clear icon should be 75% of the size in MobileFrontend search
Closed, ResolvedPublic0.5 Estimated Story Points

Description

Per @Nirzar

  • Introduce -small modifier to mediawiki ui icon in core (Fixed in Ia140779d488dddf9014087e8d48852c8c53977e3)
  • Clear icon in search should add the mw-ui-icon-small modifier
  • The RHS padding of the clear icon is reduced to match that of the icon on the LHS

Screen Shot 2016-08-17 at 2.38.11 PM.png (59×905 px, 9 KB)

Event Timeline

Jdlrobson renamed this task from Clear icon should be 75% of the size to Clear icon should be 75% of the size in MobileFrontend search.Aug 17 2016, 9:38 PM
Jdlrobson added projects: MobileFrontend, Design.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a subscriber: Nirzar.
Jdlrobson added a subscriber: Volker_E.
jhobs added a subscriber: jhobs.

@Nirzar please put whatever priority you think is appropriate. And to be clear, this is to be applied to MF to override core's styles, not to be applied to core, correct?

@Nirzar, would you please add a "As an x I want y so that z" to this?

Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: Technical-Debt.

@dr0ptp4kt no need. This is a technical debt task.

In Triaged but Future, but no priority. Got one?

Nirzar triaged this task as Medium priority.Aug 25 2016, 8:56 PM

Change 306796 had a related patch set uploaded (by Jdlrobson):
Clear icon should be small

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

Change 306796 merged by jenkins-bot:
Clear icon should be small

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

This LGTM.

Tested in the following browsers:

  • Chrome (52.0.2743.116) on OS X El Capitan (10.11.6)
  • Firefox (47.0.1) on OS X El Capitan (10.11.6)
  • Safari (9.1.2) on OS X El Capitan (10.11.6)
  • Edge 14 on Windows 10
  • Windows Phone 8.1 on an emulated Nokia Lumia 930
  • Chrome (49.0.2623.91) on a Galaxy S7
  • Chrome (49.0.2623.91) on a Galaxy Tab 4 10.1

Screen Shot 2016-08-26 at 10.42.20.png (46×74 px, 7 KB)

@Nirzar: Now that the icon has been made smaller should we reduce the padding on the RHS accordingly?

@Nirzar: Now that the icon has been made smaller should we reduce the padding on the RHS accordingly?

YES! thank you. top right bottom should be the same space. you can adjust the padding accordingly.

Should that be in a follow on task? If so, @Nirzar, would you please sign off on this task and close resolved if appropriate?

Should that be in a follow on task? If so, @Nirzar, would you please sign off on this task and close resolved if appropriate?

@Nirzar: Gimme a number and I'll submit a patch. It's such a trivial change that creating a follow-on task for it seems frivolous and we run the risk of missing the next branch cut.

Sorry, read your comment after i created the subtask. i marked the subtask invalid.

here's the value

.search-overlay .clear {
right: 0.5em;
}

Change 307052 had a related patch set uploaded (by Phuedx):
Search Overlay: Reduce clear icon's right gutter

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

Change 307052 merged by jenkins-bot:
Search Overlay: Reduce clear icon's right gutter

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

MBinder_WMF set the point value for this task to 0.5.Aug 29 2016, 4:03 PM

@dr0ptp4kt Can we resolve this? Otherwise it needs to be carried over to Reading-Web-Sprint-80-V-for-Vandalism

I defer to @Nirzar. @Nirzar, would you please mark Resolved if it's Done?