Add link text to ProcessPageEditLink

[tpr]

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.

[teppo]

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

[tpr]

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.

[tpr]

Created a PR:

https://github.com/processwire/processwire/pull/38