NPM Resource Changed ? Pending

User tests: Successful: Unsuccessful:

avatar vorayash
vorayash
3 May 2022

Pull Request for Issue #37593

Summary of Changes

  1. To select chevrons, 'i' was used instead of 'span' because of that it was not selecting anything.

Testing Instructions

Go to Modules >> any module >> Menu Assignment tab >> Only on the pages selected
Click on one or two menus to expand them
The chevrons on the left change from .icon-chevron-right to .icon-chevron-down correctly
Now click on >> Expand the Menu Subtrees >> None

Actual result BEFORE applying this Pull Request

The chevrons do not change so some are incorrect

Expected result AFTER applying this Pull Request

The chevrons change correctly.
1go1wg5NDk

90ecacf 22 Jan 2022 avatar laoneo cs
50a8824 26 Jan 2022 avatar brianteeman tweak
avatar vorayash vorayash - open - 3 May 2022
avatar vorayash vorayash - change - 3 May 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 May 2022
Category JavaScript Repository NPM Change
avatar brianteeman
brianteeman - comment - 3 May 2022

This is a bugfix so andnot a new feature so really should be rebased on 4.1

avatar vorayash
vorayash - comment - 3 May 2022

So I am supposed to make pull request on 4.1, right?

avatar brianteeman
brianteeman - comment - 3 May 2022

you should be able to just rebase it without creating a new pr.

rebase

avatar vorayash vorayash - change - 3 May 2022
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2022-05-03 15:32:53
Closed_By vorayash
Labels Added: NPM Resource Changed ?
avatar vorayash vorayash - close - 3 May 2022
avatar joomla-cms-bot joomla-cms-bot - change - 3 May 2022
Category JavaScript Repository NPM Change Unit Tests Repository Administration com_admin SQL Postgresql
avatar vorayash
vorayash - comment - 3 May 2022

Closed due to conflicts.

Add a Comment

Login with GitHub to post a comment