NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar Quy
Quy
31 Jul 2020

Pull Request for Issue #29335 .

Summary of Changes

Add padding to external link icon.

In Help Dashboard, padding is slightly wider due to markup, but at least it is better with this PR than before.

Testing Instructions

Run npm run build:css or download the installer package at the bottom of the page.

Navigate backend to see external links in:

  • Post Installation Messages
  • reCAPTCHA plugin
  • Two Factor Authentication - Google Authenticator plugin
  • Content Security Policy configuration and select Custom (see screenshots)
  • Extensions > Install from Web
  • Help Dashboard

Actual result BEFORE applying this Pull Request

29335-before

Expected result AFTER applying this Pull Request

29335-before

avatar Quy Quy - open - 31 Jul 2020
avatar Quy Quy - change - 31 Jul 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Jul 2020
Category Administration Templates (admin) NPM Change
avatar Quy Quy - change - 31 Jul 2020
Labels Added: NPM Resource Changed ?
avatar Quy Quy - change - 31 Jul 2020
The description was changed
avatar Quy Quy - edited - 31 Jul 2020
avatar sandewt
sandewt - comment - 1 Aug 2020

Test results: OK = successfully

Post Installation Messages OK
reCAPTCHA plugin OK
Two Factor Authentication - Google Authenticator plugin OK
Content Security Policy configuration and select Custom (see screenshots) OK
Extensions > Install from Web OK

Help Dashboard OK / NOK !?

Before testing: already a little space
screen shot 2020-08-01 at 11 52 35

After testing: just a little more space
screen shot 2020-08-01 at 11 52 35


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

avatar chmst chmst - test_item - 1 Aug 2020 - Tested successfully
avatar chmst
chmst - comment - 1 Aug 2020

I have tested this item successfully on 2a1962c

Better than before, especially on the install-from-web.


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

avatar HLeithner
HLeithner - comment - 1 Aug 2020
avatar hans2103
hans2103 - comment - 1 Aug 2020

@Quy
change the position of padding-right.

administrator/templates/atum/scss/blocks/_global.scss
--
4 | 175:3  ✖  Expected "padding-right" to come before "content"   order/properties-order
18fb6a2 1 Aug 2020 avatar Quy cs
avatar Quy
Quy - comment - 1 Aug 2020

@sandewt @hans2103 Please test as it. The Help Dashboard and several other places can be fixed in a separate PR since changes will be more extensive. Thanks.

avatar HLeithner HLeithner - change - 1 Aug 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-08-01 22:57:12
Closed_By HLeithner
avatar HLeithner HLeithner - close - 1 Aug 2020
avatar HLeithner HLeithner - merge - 1 Aug 2020
avatar HLeithner
HLeithner - comment - 1 Aug 2020

Thanks

Add a Comment

Login with GitHub to post a comment