RTC NPM Resource Changed PR-5.1-dev Pending

User tests: Successful: Unsuccessful:

avatar obuisard
obuisard
14 Mar 2024

Summary of Changes

shepherd-button-primary button needs to have a border always.
Replaced 'border-color' properties with 'border'.
It also ensures buttons for the tour are consistent to the buttons of the rest of the interface (in dark mode especially, where borders are more prominent).

Testing Instructions

Get the latest 5.1 version.
Start a tour.
Hover the 'Start' button.

Actual result BEFORE applying this Pull Request

The Start button gets a border on hover, while the initial button does not have a border, resulting on a certain flicker.

Expected result AFTER applying this Pull Request

No flicker, the primary button looks like the rest of the primary buttons on the page.

Link to documentations

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 obuisard obuisard - open - 14 Mar 2024
avatar obuisard obuisard - change - 14 Mar 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Mar 2024
Category Repository NPM Change
avatar Quy Quy - test_item - 14 Mar 2024 - Tested successfully
avatar Quy
Quy - comment - 14 Mar 2024

I have tested this item ✅ successfully on 17f45b7


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

avatar obuisard obuisard - change - 14 Mar 2024
Title
Tour primary button changes size on hover, adding annoying flicker
[5.1] Tour primary button changes size on hover, adding annoying flicker
avatar obuisard obuisard - edited - 14 Mar 2024
avatar viocassel viocassel - test_item - 15 Mar 2024 - Tested successfully
avatar viocassel
viocassel - comment - 15 Mar 2024

I have tested this item ✅ successfully on 17f45b7


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

avatar Quy Quy - change - 15 Mar 2024
Status Pending Ready to Commit
Labels Added: NPM Resource Changed PR-5.1-dev
avatar Quy
Quy - comment - 15 Mar 2024

RTC


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

avatar garstud garstud - test_item - 15 Mar 2024 - Tested successfully
avatar garstud
garstud - comment - 15 Mar 2024

I have tested this item ✅ successfully on 17f45b7


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

avatar Quy Quy - change - 15 Mar 2024
Labels Added: RTC
avatar bembelimen bembelimen - change - 17 Mar 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-03-17 14:54:15
Closed_By bembelimen
avatar bembelimen bembelimen - close - 17 Mar 2024
avatar bembelimen bembelimen - merge - 17 Mar 2024

Add a Comment

Login with GitHub to post a comment