? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
13 Jun 2016

The JED Install message box has a X in the top right corner which I keep clicking on to CLOSE the message. But its a link to the options and not a close.

This cosmetic PR changes it from a X to remove confusion

Before

pgql

After

25tv

avatar brianteeman brianteeman - open - 13 Jun 2016
avatar brianteeman brianteeman - change - 13 Jun 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 Jun 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 13 Jun 2016
Category Templates (admin) UI/UX
avatar piotr-cz
piotr-cz - comment - 14 Jun 2016

Thanks, the x button is misleading

avatar brianteeman
brianteeman - comment - 14 Jun 2016

Can you test it and mark the results please

avatar bertmert bertmert - test_item - 14 Jun 2016 - Tested successfully
avatar bertmert
bertmert - comment - 14 Jun 2016

I have tested this item successfully on 95aa152


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar piotr-cz
piotr-cz - comment - 15 Jun 2016

what about using am info sign (i) or question mark sign (?) icons

avatar andrepereiradasilva
andrepereiradasilva - comment - 15 Jun 2016

since you'll go to options panel, the icon i think it should be is the options icons (the same that exists in options button).

avatar brianteeman
brianteeman - comment - 15 Jun 2016

I'm open to any suggestion for the symbol displayed. As far as I can tell
it must be an html entity though
On 15 Jun 2016 9:06 a.m., "andrepereiradasilva" notifications@github.com
wrote:

since you'll go to options panel, the icon i think it should be is the
options icons (the same that exists in options button).


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#10811 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8VLf01zIbLio7R5r4tSg2C7CvtCFks5qL7J0gaJpZM4I0cZH
.

avatar andrepereiradasilva
andrepereiradasilva - comment - 15 Jun 2016

did you try:
<span class="icon-options"></span>

avatar brianteeman
brianteeman - comment - 16 Jun 2016

Not a fan of doing that as it would not be an accessible link


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar brianteeman
brianteeman - comment - 16 Jun 2016

@wilsonge @rolandd can you make a decision on the htmlentity to use please


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar crommie crommie - test_item - 25 Jun 2016 - Tested successfully
avatar crommie
crommie - comment - 25 Jun 2016

I have tested this item successfully on 95aa152

As expected.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar pe7er pe7er - test_item - 25 Jun 2016 - Tested successfully
avatar pe7er
pe7er - comment - 25 Jun 2016

I have tested this item successfully on 95aa152


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar roland-d
roland-d - comment - 25 Jun 2016

@brianteeman I agree with Andre that we should use the same symbol as for the Options. To get this done you can remove the &oplus; and leave the string empty. In the class statement add icon-options and you will get the symbol used to go to Options. That should be good enough I think.

avatar brianteeman
brianteeman - comment - 25 Jun 2016

Will that work in Hathor?

avatar roland-d
roland-d - comment - 25 Jun 2016

Yes, works in Hathor as well.

avatar brianteeman
brianteeman - comment - 25 Jun 2016

Ok . Am at a 40th birthday I will fix tomorrow

avatar roland-d
roland-d - comment - 25 Jun 2016

Thanks

avatar joomla-cms-bot
joomla-cms-bot - comment - 26 Jun 2016

This PR has received new commits.

CC: @bertmert, @crommie, @pe7er


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar brianteeman
brianteeman - comment - 26 Jun 2016

Updated the PR with @roland-d and @andrepereiradasilva suggestion to use the options icon as seen in this screenshot

4cfb

avatar andrepereiradasilva
andrepereiradasilva - comment - 26 Jun 2016

IMHO the icon is to big, can you remove the close class?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar brianteeman
brianteeman - comment - 26 Jun 2016

Not that simple. The close class also gives it the position

On 26 June 2016 at 12:38, andrepereiradasilva notifications@github.com
wrote:

IMHO the icon is to big, can you remove the close clsss?

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/10811
https://issues.joomla.org/tracker/joomla-cms/10811.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10811 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8feAssLa4SO5bcyQYMbf8xkGu9mJks5qPmStgaJpZM4I0cZH
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman
brianteeman - comment - 26 Jun 2016

I guess I could give it its own class
alert-options

?

On 26 June 2016 at 12:50, Brian Teeman brian@teeman.net wrote:

Not that simple. The close class also gives it the position

On 26 June 2016 at 12:38, andrepereiradasilva notifications@github.com
wrote:

IMHO the icon is to big, can you remove the close clsss?

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/10811
https://issues.joomla.org/tracker/joomla-cms/10811.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#10811 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8feAssLa4SO5bcyQYMbf8xkGu9mJks5qPmStgaJpZM4I0cZH
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar joomla-cms-bot
joomla-cms-bot - comment - 27 Jun 2016

This PR has received new commits.

CC: @bertmert, @crommie, @pe7er


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar conconnl conconnl - test_item - 29 Jun 2016 - Tested successfully
avatar conconnl
conconnl - comment - 29 Jun 2016

I have tested this item successfully on 058362e


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar nikitadhiman nikitadhiman - test_item - 30 Jun 2016 - Tested successfully
avatar nikitadhiman
nikitadhiman - comment - 30 Jun 2016

I have tested this item successfully on 058362e


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar zero-24 zero-24 - change - 30 Jun 2016
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 30 Jun 2016

RTC :)


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar joomla-cms-bot joomla-cms-bot - change - 30 Jun 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 8 Jul 2016
Milestone Added:
avatar roland-d
roland-d - comment - 16 Jul 2016

@brianteeman The new icon is not in the same location as the old icon. The new icon is quite a lot close to the text. Is that on purpose?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar brianteeman
brianteeman - comment - 16 Jul 2016

The icon is inside the div and the div has been given a margin - I dont think it can be altered when using an icon as requested


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10811.

avatar roland-d
roland-d - comment - 16 Jul 2016

I have no preference either way, just noticed the difference. On one hand I am thinking it is OK because it is not your regular close button. If someone wants to change it later, it can always be done. The change of icon is more important than it's location.

avatar roland-d roland-d - change - 16 Jul 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-07-16 09:31:21
Closed_By roland-d
avatar roland-d roland-d - close - 16 Jul 2016
avatar roland-d roland-d - merge - 16 Jul 2016
avatar joomla-cms-bot joomla-cms-bot - close - 16 Jul 2016
avatar roland-d roland-d - reference | aca87a7 - 16 Jul 16
avatar roland-d roland-d - merge - 16 Jul 2016
avatar roland-d roland-d - close - 16 Jul 2016
avatar joomla-cms-bot joomla-cms-bot - change - 16 Jul 2016
Labels Removed: ?
avatar brianteeman
brianteeman - comment - 16 Jul 2016

Thanks

avatar brianteeman brianteeman - head_ref_deleted - 16 Jul 2016

Add a Comment

Login with GitHub to post a comment