Feeling Lucky
?
?
[#15528] - Move discover message and add style
- Fixed in Code Base
- 1 May 2017
- Medium
- Build: staging
- # 15528
- Diff
- brianteeman:discover
User tests: Successful: Unsuccessful:
Pull Request for Issue #15507 .
Summary of Changes
Move message to above the list to be more consistent and add standard styling
After PR
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_installer |
infograf768
- comment
- 25 Apr 2017
The way this patch is done, the message would show only when there are already extensions to discover in the list and not permanently as it was designed to be.
| Labels |
Added:
?
|
||
infograf768
- comment
- 25 Apr 2017
imho
<div class="alert alert-no-items alert-info">
<?php echo JText::_('COM_INSTALLER_MSG_DISCOVER_DESCRIPTION'); ?>
</div>
should go before
<?php if (empty($this->items)) : ?>
brianteeman
- comment
- 25 Apr 2017
infograf768
- comment
- 25 Apr 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15528.
franz-wohlkoenig
- comment
- 25 Apr 2017
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15528.
| Status | Pending | ⇒ | Ready to Commit |
franz-wohlkoenig
- comment
- 25 Apr 2017
RTC after two successful tests.
rdeutz
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-05-01 20:42:55 |
| Closed_By | ⇒ | rdeutz | |
| Labels |
Added:
?
|
||
brianteeman
- comment
- 1 May 2017
Thanks
I have tested this item? unsuccessfully on 3ec3f84
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15528.