Last modified: 2014-02-12 23:53:34 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T48804, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 46804 - Inconsistent button sizing
Inconsistent button sizing
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
stable (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Juliusz Gonera
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-02 18:18 UTC by Munaf Assaf
Modified: 2014-02-12 23:53 UTC (History)
10 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Login Page Button (correct) (67.47 KB, image/png)
2013-04-02 18:18 UTC, Munaf Assaf
Details
Upload Image Buttons (incorrect) (82.10 KB, image/png)
2013-04-02 18:18 UTC, Munaf Assaf
Details

Description Munaf Assaf 2013-04-02 18:18:28 UTC
Created attachment 12023 [details]
Login Page Button (correct)

Button heights/paddings vary between pages on Wikipedia mobile. See the attached screenshots.

They should be standardized to match those on the Login Page (height: 39px).
Comment 1 Munaf Assaf 2013-04-02 18:18:56 UTC
Created attachment 12024 [details]
Upload Image Buttons (incorrect)
Comment 2 Jon 2013-04-17 16:49:10 UTC
Does this just effect the photo uploads button?
Comment 3 Munaf Assaf 2013-04-17 17:18:50 UTC
That's the only place I saw it, though you could argue that the button in the Settings page is also inconsistent.

We should just grep for all buttons in MobileFrontend and make sure they are all using the exact same class. It will probably be a code reduction because we can delete whatever redundant classes are there.
Comment 4 Maryana Pinchuk 2013-04-17 17:58:10 UTC
The "confirm settings" button is kinda wonky (especially the fact that it's grayed out, which makes it look inactionable) and doesn't resemble any of our other buttons. It would be nice if it were made consistent with login, upload submit/cancel, etc.

And while we're at it, can we change the settings opt in/out buttons, too? They look like toggles but aren't; I've seen several users try to drag them instead of tapping. Also, there's a weird split-second flash of check-marks before they fully load.
Comment 5 Jon 2013-04-17 18:04:26 UTC
The reason I ask is that all buttons are styled the same but it's possible certain buttons are adding additional styling which is what needs to be removed...

Maryana - I fixed the toggle issue [1] on the settings checkboxes (they are not buttons :)) a while back but that's an orthogonal discussion. The split second flash is also a known bug (bug 46906) which also has a fix.

Can you open a separate bug for colour of settings page save button?

[1] https://gerrit.wikimedia.org/r/#/c/59013/
Comment 6 Maryana Pinchuk 2013-04-17 19:10:07 UTC
https://bugzilla.wikimedia.org/show_bug.cgi?id=47330 for button color, thanks to Munaf :)

I'm still seeing the toggle flashes – did those patches not get deployed? If not, I can make a separate story/bug for this, to make sure we prioritize taking care of it.
Comment 7 Gerrit Notification Bot 2013-05-03 16:58:10 UTC
Related URL: https://gerrit.wikimedia.org/r/62166 (Gerrit Change I64c8d625b8332588316fc30eecd26b40ada13d46)
Comment 8 Jon 2013-05-03 17:04:00 UTC
https://gerrit.wikimedia.org/r/62166
Comment 9 Jon 2013-05-03 17:32:06 UTC
Munaf - Are you able to be clearer what the bug is here? I reset the padding for the photo upload buttons to be exactly the same as the inputs on the login form but Juliusz has complained they are wrong (see https://gerrit.wikimedia.org/r/#/c/62166/)

What exactly should be padding left/right/top/bottom ?
Comment 10 Gerrit Notification Bot 2013-05-13 21:32:46 UTC
https://gerrit.wikimedia.org/r/62166 (Gerrit Change I64c8d625b8332588316fc30eecd26b40ada13d46) | change APPROVED and MERGED [by Jdlrobson]

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links