Posted by sun on November 4, 2009 at 3:39am
Jump to:
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | system.module |
| Category: | bug report |
| Priority: | normal |
| Assigned: | sun |
| Status: | closed (fixed) |
| Issue tags: | Contextual links |
Issue Summary
A module that wants to provide a different behavior and representation for contextual links will have a performance impact, because the JS/CSS cannot be altered once.
Just make it a library and the issue is resolved.
I already had this in mind when doing the original implementation, but somehow, I forgot.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| drupal.contextual-links-library.patch | 1.2 KB | Idle | Passed: 14679 passes, 0 fails, 0 exceptions | View details |
Comments
#1
Simple and elegant. Like sun said, having this allows contrib modules to change how the contextual links look/work.
#2
I want this to get in before doing #626286: Make contextual links a module
#3
How can you forget? Jeez :'D
#4
Makes sense to me.
Committed to HEAD.
#5
Automatically closed -- issue fixed for 2 weeks with no activity.