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
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 3120327_6.patch | 4.35 KB | vsujeetkumar |
| #3 | 3120327-3-extlink-1.2.patch | 4.09 KB | mbovan |
| #2 | 3120327-2.patch | 4.25 KB | mbovan |
Comments
Comment #2
mbovan commentedWorking patch with tests to implement suggestions from the issue summary.
Comment #3
mbovan commentedThis is a patch based on the latest stable release 8.x-1.2.
Comment #4
neslee canil pinto@mbovan, #3 doesn't apply, needs a reroll. Thanks
Comment #5
vsujeetkumar commentedComment #6
vsujeetkumar commentedRe-roll patch created, Please review.
Comment #7
johnchqueComment #8
neslee canil pintoComment #11
neslee canil pintoCommitted, Thanks.