[
https://issues.jboss.org/browse/JBIDE-14735?page=com.atlassian.jira.plugi...
]
Victor Rubezhny commented on JBIDE-14735:
-----------------------------------------
We cannot "just trim the string" because there are many other existing snippets
in other palettes that use the same properties for insertion. If we change the insertion
rules we have to test and modify all those snippets if this insertion rule change breaks
their work.
For this certain snippet we should remove the ending '/n'-char at the end due to
make it work as you described above. But not change the insertion rule.
Improve text formatting when dropping widgets from Palette view
---------------------------------------------------------------
Key: JBIDE-14735
URL:
https://issues.jboss.org/browse/JBIDE-14735
Project: Tools (JBoss Tools)
Issue Type: Enhancement
Components: jsp/jsf/xml/html source editing
Reporter: Alexey Kazakov
Assignee: Victor Rubezhny
Labels: new_and_noteworthy
Fix For: 4.2.x
1. Suppose we have the following code:
{code}
<div data-role="page" id="page-1">
<div data-role="content">|
</div>
</div>
{code}
| marks the cursor position.
2. Press Enter right after <div data-role="content">:
{code}
<div data-role="page" id="page-1">
<div data-role="content">
|
</div>
</div>
{code}
3. Then click on *button* widget in jQuery Mobile Palette. Click on OK.
The following code is generated:
{code}
<div data-role="page" id="page-1">
<div data-role="content">
<a href="" id="button-1" data-role="button">Link
button</a>
</div>
</div>
{code}
But it should be
{code}
<div data-role="page" id="page-1">
<div data-role="content">
<a href="" id="button-1" data-role="button">Link
button</a>
</div>
</div>
{code}
instead.
Also we probably should use the preferences/settings of the editor to insert space
symbols (N spaces/tab) instead of hardcoded " ".
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:
http://www.atlassian.com/software/jira