[5.x] Feature/phone and mail link #10292
Conversation
j3ll3yfi5h
changed the title
There was a problem hiding this comment.
You've added handling for mailto links, but nothing for tel links.
When you pick phone in the option dropdown, there's no way for you to enter a value.
When you choose the mail option, with every keystroke it adds mailto: to the start.
Also, please disable your code formatter while you work on this so you don't end up committing unrelated changes. Thanks!
|
I'm going to close this for now to keep things organized. If/when you're ready to continue just let us know and we can reopen this. |
|
FYI: I've been working with Erin on this at Flat Camp and just wanted to know if you generally welcome this PR or if it will be rejected anyway. That's why it was not fully finished. Thanks for the feedback. As soon as I get time for it again, I'll let you know. |
|
This PR would be welcomed! |
Closes statamic/ideas#1174