NPM Resource Changed PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar sandewt
sandewt
13 Jul 2023

Pull Request for Issue # .

Summary of Changes

The arrow-up icon of the Back-to-top Link should be visible on hovering.
And in addition, the color of the button must then be blue.

Related to the Button text color issue #40435, that caused this problem.

Testing Instructions

Build npm ci.

Enable the Back-to-top Link button in Templates: Edit Style.

Hover on the Back-to-top button in the fontend of the site.

Code review

Actual result BEFORE applying this Pull Request

The arrow-up icon is NOT visible on hovering on the button.

Schermafbeelding 2023-07-13 115902

Expected result AFTER applying this Pull Request

The arrow-up icon is visible on hovering on the button.

Schermafbeelding 2023-07-13 120207

Link

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 13 Jul 2023
Category Repository NPM Change
avatar sandewt sandewt - open - 13 Jul 2023
avatar sandewt sandewt - change - 13 Jul 2023
Status New Pending
avatar sandewt sandewt - change - 13 Jul 2023
The description was changed
avatar sandewt sandewt - edited - 13 Jul 2023
avatar brianteeman
brianteeman - comment - 13 Jul 2023

I cannot replicate the problem on j43 or j5

avatar sandewt
sandewt - comment - 13 Jul 2023

I cannot replicate the problem on j43

Clear the browser cache.

avatar brianteeman
brianteeman - comment - 13 Jul 2023

How will clearing the browser cache have any impact on the site BEFORE applying this PR?

I have not applied the PR because I can not replicate the reported problem

avatar sandewt
sandewt - comment - 13 Jul 2023

How will clearing the browser cache have any impact on the site BEFORE applying this PR?

What is the result if you clear the browser history?

avatar brianteeman
brianteeman - comment - 13 Jul 2023

it is the same of course

avatar sandewt
sandewt - comment - 13 Jul 2023

it is the same of course

I should have known better.

System information:
Windows 11, Firefox, MS Edge and Google Chrome.

avatar brianteeman
brianteeman - comment - 19 Jul 2023

Found it and the reason why I couldnt replicate.

The problem is from your pull request #40435

As that hasnt been upmerged into 4.4 or 5 yet then I didnt have the faulty css

avatar obuisard
obuisard - comment - 14 Aug 2023

Not a fan of adding !important in the core, as it makes CSS overrides difficult down the road. Better complement #40435 to fix issues resulting from it.

avatar jackal-kr
jackal-kr - comment - 16 Aug 2023

Hello,

I had a similar issue on my website after upgrading to 4.3.3 and had to roll-back to 4.3.2 because it broke the hover colors not only of the "Back-to-top Link" but of some other components as well. Took me a while to understand that was not me but the upgrade. Restored from backup to 4.3.2 and all was fixed. Simulated on my laptop and yes that was the culprit -> 4.3.3..

Any news when it will be fixed? Any workaround meanwhile?..

Thanks a lot!


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

avatar obuisard
obuisard - comment - 17 Aug 2023

The main issue comes from the fact that many button-like elements are actually not using the .btn class and that is where the confusion starts... So any 'a' element, styled as a button, but not having the .btn class will have a problem

avatar sandewt
sandewt - comment - 26 Aug 2023
avatar ChristineWk
ChristineWk - comment - 2 Sep 2023

Issue confirmed (Back to top Link) but NPM Package is not available (404)

avatar sandewt
sandewt - comment - 2 Sep 2023

Issue confirmed (Back to top Link) but NPM Package is not available (404)

You can still download the package; or in the command-line interface run the npm ci command.

Anyway, I prefer the overall solution, which I have proposed here: #40435 (comment)
However, I am still waiting for a response(s).

avatar sandewt
sandewt - comment - 14 Sep 2023

Closed because PR #41749

avatar sandewt sandewt - change - 14 Sep 2023
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2023-09-14 11:48:08
Closed_By sandewt
Labels Added: NPM Resource Changed PR-4.3-dev
avatar sandewt sandewt - close - 14 Sep 2023

Add a Comment

Login with GitHub to post a comment