?

User tests: Successful: Unsuccessful:

avatar N6REJ
N6REJ
27 Oct 2014

@phproberto @betweenbrain Hopefully this is right his time!

avatar N6REJ N6REJ - open - 27 Oct 2014
avatar jissues-bot jissues-bot - change - 27 Oct 2014
Labels Added: ?
avatar N6REJ N6REJ - change - 27 Oct 2014
Title
created new branch and applied diff
Make icon spacing consistent based on .css
avatar N6REJ N6REJ - close - 27 Oct 2014
avatar N6REJ N6REJ - change - 27 Oct 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-10-27 19:32:37
avatar N6REJ N6REJ - reopen - 27 Oct 2014
avatar N6REJ N6REJ - change - 27 Oct 2014
Status Closed New
avatar N6REJ
N6REJ - comment - 28 Oct 2014

Passed TRAVIS and ready for testing.

avatar brianteeman brianteeman - change - 28 Oct 2014
Category Templates (admin) Templates (site) UI/UX
avatar brianteeman brianteeman - change - 28 Oct 2014
Status New Pending
avatar jamgit
jamgit - comment - 1 Nov 2014

How can this be tested, please describe how I can help.

avatar N6REJ
N6REJ - comment - 2 Nov 2014

@jamgit sorry I forgot to do that when I had to change pr's... so, to test, look @ the icons, such as edit, in any/all views and pre-patch you will see spaces, sometimes before , sometimes after also. This can best be viewed in the inspector as shown in these 2 images. After patch the visual space should still remain but no hardcoded spaces should exist anymore. Everything should be lined up properly vertically as well.
icon-after
icons

avatar infograf768
infograf768 - comment - 5 Nov 2014

@test
Looks OK here

avatar infograf768
infograf768 - comment - 5 Nov 2014

@N6REJ

You will have though to regenerate css as there was a mistake in a PR which I just corrected
51ed8a5

avatar infograf768
infograf768 - comment - 6 Nov 2014

lucky you, the PR still applies

avatar N6REJ
N6REJ - comment - 6 Nov 2014

yay! lol
Bear
On 11/6/2014 01:44, infograf768 wrote:

lucky you, the PR still applies


Reply to this email directly or view it on GitHub
#4941 (comment).

No virus found in this message.
Checked by AVG - www.avg.com http://www.avg.com
Version: 2015.0.5315 / Virus Database: 4189/8521 - Release Date: 11/06/14

avatar N6REJ N6REJ - close - 23 Dec 2014
avatar N6REJ N6REJ - change - 23 Dec 2014
Status Pending Closed
Closed_Date 2014-10-27 19:32:37 2014-12-23 18:27:16

Add a Comment

Login with GitHub to post a comment