Skip to content

Conversation

@infograf768
Copy link
Member

Test: display the Permissions tab for any component or global.

Before this patch the title and the content of the tip are incorrectly displayed this way:

screen shot 2015-10-26 at 08 38 03

After patch, we get:

screen shot 2015-10-26 at 08 36 50

When the PR #8150 will be merged, we will get:

screen shot 2015-10-26 at 09 09 08

Note: htmlspecialchars is already modified in #8150

@fontanil
Copy link

@test
Perfect for me. Thanks!

@fontanil
Copy link

@test
I confirm. Test OK


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

@ghost
Copy link

ghost commented Oct 26, 2015

@test Tested successfully ✅ this PR in combination with #8150

@zero-24 zero-24 added this to the Joomla! 3.5.0 milestone Oct 26, 2015
@zero-24
Copy link
Contributor

zero-24 commented Oct 26, 2015

RTC for 3.5.0 Thanks.


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Oct 26, 2015
@infograf768
Copy link
Member Author

@zero-24 this pr corrects a bug and is not related to the other one. i suggest to tag it to 3.4.6

@zero-24 zero-24 modified the milestones: Joomla! 3.4.6, Joomla! 3.5.0 Oct 26, 2015
@zero-24
Copy link
Contributor

zero-24 commented Oct 26, 2015

done i have read your comment in the wrong way ;)

rdeutz added a commit that referenced this pull request Oct 26, 2015
Correcting Permissions tooltips display
@rdeutz rdeutz merged commit baaaaa6 into joomla:staging Oct 26, 2015
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Oct 26, 2015
@zero-24 zero-24 modified the milestones: Joomla! 3.4.6, Joomla! 3.5.0 Oct 28, 2015
@fontanil
Copy link

fontanil commented Feb 5, 2016

I have tested this item ✅ successfully on 6518643

Successfully tested on last staging. Thanks!


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants