? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
22 Jun 2015

In the article manage when selecting a User in the publishing tab the list of users is displayed in a modal window.
When filtering and if no results, the display is just empty instead of the usual message "No Matching Results"
This patch is just implementing the message in the modal when necessary.

Thanks to @infograf768 for the code idea

avatar brianteeman brianteeman - open - 22 Jun 2015
avatar brianteeman brianteeman - change - 22 Jun 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jun 2015
Labels Added: ?
avatar brianteeman brianteeman - change - 22 Jun 2015
Category Administration UI/UX
avatar brianteeman brianteeman - change - 22 Jun 2015
Easy No Yes
avatar infograf768
infograf768 - comment - 22 Jun 2015

OK here.


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

avatar infograf768 infograf768 - alter_testresult - 22 Jun 2015 - infograf768: Tested successfully
avatar Kubik-Rubik
Kubik-Rubik - comment - 23 Jun 2015

@brianteeman You have to move all lines in the code section one indent to the right, so that the opening and ending tags are in the same row (e.g. thead is not in the same row). Thanks!

avatar brianteeman
brianteeman - comment - 23 Jun 2015

Sorry not got a clue what you mean
Please create a PR against my branch so that I can see what you mean
(BTW I think you are confusing row and column)

On 23 June 2015 at 12:36, Viktor Vogel notifications@github.com wrote:

@brianteeman https://github.com/brianteeman You have to move all lines
in the code section one indent to the right, so that the opening and ending
tags are in the same row (e.g. thead is not in the same row). Thanks!


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

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

avatar Kubik-Rubik
Kubik-Rubik - comment - 23 Jun 2015

@brianteeman Sorry, I meant column of course. Okay, I will send you a PR soon. :-)

avatar Kubik-Rubik
Kubik-Rubik - comment - 23 Jun 2015

Thank you, @brianteeman and @infograf768!


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

avatar Kubik-Rubik Kubik-Rubik - test_item - 23 Jun 2015 - Tested successfully
avatar Kubik-Rubik Kubik-Rubik - change - 23 Jun 2015
Status Pending Ready to Commit
avatar Kubik-Rubik Kubik-Rubik - change - 23 Jun 2015
Labels Added: ?
avatar Kubik-Rubik
Kubik-Rubik - comment - 3 Jul 2015

Thank you @brianteeman! Merged.

avatar zero-24 zero-24 - close - 3 Jul 2015
avatar brianteeman brianteeman - change - 3 Jul 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-07-03 12:34:20
Closed_By brianteeman
avatar brianteeman brianteeman - close - 3 Jul 2015
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment