Problem/Motivation

Steps to reproduce

  1. Visit any view, Exp: /admin/structure/views/view/articles_aside
  2. Add/remove any field and open the dialog and check the buttons on hover.

Proposed resolution

Jquery.ui is overriding the CSS changes, so add border and font-weight on hover, focus and active state.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Adding before patch screen recording for reference.

CommentFileSizeAuthor
#6 3386370.mov10.54 MBnilesh.k
#4 screen-recording.mp4166.7 KBshweta__sharma
Screen Recording 2023-09-11 at 9.12.55 AM.mov1.56 MBgauravvvv

Issue fork drupal-3386370

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Gauravvvv created an issue. See original summary.

Gauravvvv opened merge request !4740

gauravvvv’s picture

gauravvvv commented
Issue summary: View changes
Status: Active » Needs review
shweta__sharma’s picture

shweta__sharma commented
StatusFileSize
new screen-recording.mp4166.7 KB

Hi @Gauravvvv
I have checked the mentioned issue on my local but I am not able to reproduce the issue which you mentioned. See the reference video and let me know if I am missing any steps. Followed above mentioned steps.
Thanks

preeti.chawla’s picture

preeti.chawla commented

Hi @Gauravvvv
I have checked with Claro theme, button are working fine as expected. Please add more details to reproduce it

nilesh.k’s picture

nilesh.k commented
StatusFileSize
new 3386370.mov10.54 MB

Hi @Gauravvvv, I followed the steps you provided to check the issue, but I'm unable to reproduce it. I followed these steps:

Created a new view.
Added it to the filter criteria.
Opened that filter criteria.
Then, I removed it and added another filter criteria.
Please let me know if there's anything else I should try or if you have any additional information. Also I have attached a video for more information. Please take a look.

smustgrave’s picture

smustgrave commented
Status: Needs review » Postponed (maintainer needs more info)

@Gauravvvv can you provide more detail seems others are not able to reproduce

gauravvvv’s picture

gauravvvv commented
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

I checked it, this was because of old vendor directory. I was on Drupal 11 but the vendor directory was of Drupal 10. We can close it. thanks