tpr Posted November 18, 2016 Share Posted November 18, 2016 Yesterday I got frustrated that it's hard to rename existing link that I looked into ProcessPageEditLink module and pwlink CKEditor plugin and added a "Link text" feature. The modal window doesn't have a field for this that I've added now: Anyone knows why it's missing there? Judging from the code it seems it was intentionally left out. 6 Link to comment Share on other sites More sharing options...
teppo Posted November 18, 2016 Share Posted November 18, 2016 (edited) 8 minutes ago, tpr said: Anyone knows why it's missing there? Judging from the code it seems it was intentionally left out. If there really is a reason for this, I would seriously question it's sensibility. This should really be changed in the core. Care to create a pull request or feature suggestion at GitHub? Edit: I've just moved this topic to the Wishlist & Roadmap section of the forum. The Modules area is mainly intended for module release / support threads, and while this question is technically related to a module, it's a core module with no support board of it's own. Edited November 18, 2016 by teppo 2 Link to comment Share on other sites More sharing options...
tpr Posted November 18, 2016 Author Share Posted November 18, 2016 Sorry for the wrong topic. Yes, my goal is to create a PR, just wanted to know if there's something I should know before. This seems a trivial feature to me, wonder why it was not added so far. 2 Link to comment Share on other sites More sharing options...
tpr Posted November 22, 2016 Author Share Posted November 22, 2016 Created a PR: https://github.com/processwire/processwire/pull/38 3 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now