Problem/Motivation

Combined with feature requests #3228882: Ignore AJAX links, #3228966: Do not lock the screen if the link has target attribute, and the bug #3038670: Annoying load of spinner on Drupal 8 Admin toolbar use, let's ignore more links.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 interdiff-3-4.txt795 bytesjungle
#4 3272476-4.patch1.42 KBjungle
#3 3272476-3.patch1.25 KBjungle

Comments

jungle created an issue. See original summary.

jungle’s picture

jungle
He/Him
Location Chongqing, China
commented
Title: Ignore more use cases » Do not lock the screen for special links
Issue summary: View changes
jungle’s picture

jungle
He/Him
Location Chongqing, China
commented
Status: Active » Needs review
StatusFileSize
new 3272476-3.patch1.25 KB

Cached $(this), ignored ajax links, ignored links with target attribute, ignore links with toolbar-item class for fixing #3038670: Annoying load of spinner on Drupal 8 Admin toolbar use

jungle’s picture

jungle
He/Him
Location Chongqing, China
commented
StatusFileSize
new 3272476-4.patch1.42 KB
new interdiff-3-4.txt795 bytes

One more for the Add element button of webform

loze’s picture

loze commented

There should be a configurable field where you can specify custom link classes to be excluded

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

It would be worth splitting off the configurability to a separate issue and committing this as is, it fixes common problems we were having with the module.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Reviewed & tested by the community
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Version: 2.0.0 » 2.0.x-dev

  • i-trokhanenko committed aa45cc75 on 2.0.x 
    Issue #3272476 by jungle, damienmckenna, loze, i-trokhanenko: Do not...
i-trokhanenko’s picture

i-trokhanenko
Location Lutsk 🇺🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.