[Wikitech-l] VisualEditor - Paste Displayed Text UI

2013-05-16 Thread Jiabao Wu
Hi

I am trying to fix bug 48489 - Can not paste new link anchor for VisualEditor.
https://bugzilla.wikimedia.org/show_bug.cgi?id=48489

I have been able to add the functionality to the link inspector to
edit the displayed text.
https://gerrit.wikimedia.org/r/#/c/64055/

I have not worked on the appearance of the interface.  What user
interface the community would like to get?

Which existing widgets if any should be included when adding this
functionality? (currently just using InputWidget.js)

Cheers,
Jiabao

___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l

Re: [Wikitech-l] VisualEditor - Paste Displayed Text UI

2013-05-16 Thread Trevor Parscal
I've already commented on the bug and in Gerrit, but I will also echo here.

First off, thank you for taking the time to work on VisualEditor, and even
submitting a patch directly into Gerrit. We hope to get more patches this
way.

In this particular case, the solution being offered isn't going to work out
for some UX and technical reasons.

Specifically that the contents of a link are not limited to plain text, so
editing them as such will remove all non-text content when the user was
only trying to change the link. The user already has a way to modify the
display content of the link, and if there are bugs to do with their ability
to do that, they should be resolved without introducing a secondary and
more limited editing affordance.

- Trevor

On Thu, May 16, 2013 at 8:00 AM, Jiabao Wu jiabao.f...@gmail.com wrote:

 Hi

 I am trying to fix bug 48489 - Can not paste new link anchor for
 VisualEditor.
 https://bugzilla.wikimedia.org/show_bug.cgi?id=48489

 I have been able to add the functionality to the link inspector to
 edit the displayed text.
 https://gerrit.wikimedia.org/r/#/c/64055/

 I have not worked on the appearance of the interface.  What user
 interface the community would like to get?

 Which existing widgets if any should be included when adding this
 functionality? (currently just using InputWidget.js)

 Cheers,
 Jiabao

 ___
 Wikitech-l mailing list
 Wikitech-l@lists.wikimedia.org
 https://lists.wikimedia.org/mailman/listinfo/wikitech-l
___
Wikitech-l mailing list
Wikitech-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikitech-l