Problem/Motivation

The extlink (JS/CSS) library is applied to all pages by default.

Usually, there is no need to apply extlink features to admin routes and it would be beneficial for performance reasons to those files at all.

Proposed resolution

Implement a new setting flag to decide whether should we apply extlink to admin routes or not. To maintain the current behaviour, enable extlink in the admin routes by default.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Comments

mbovan created an issue. See original summary.

mbovan’s picture

Status: Active » Needs review
StatusFileSize
new4.25 KB

Working patch with tests to implement suggestions from the issue summary.

mbovan’s picture

StatusFileSize
new4.09 KB

This is a patch based on the latest stable release 8.x-1.2.

neslee canil pinto’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll

@mbovan, #3 doesn't apply, needs a reroll. Thanks

vsujeetkumar’s picture

Assigned: Unassigned » vsujeetkumar
vsujeetkumar’s picture

Assigned: vsujeetkumar » Unassigned
Status: Needs work » Needs review
StatusFileSize
new4.35 KB

Re-roll patch created, Please review.

johnchque’s picture

Issue tags: -Needs reroll
neslee canil pinto’s picture

Status: Needs review » Reviewed & tested by the community

neslee canil pinto’s picture

Status: Reviewed & tested by the community » Fixed

Committed, Thanks.

Status: Fixed » Closed (fixed)

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