Bug 397711 - Setting indentation character takes no effect
Summary: Setting indentation character takes no effect
Status: REOPENED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: VJET (show other bugs)
Version: unspecified   Edit
Hardware: Macintosh Mac OS X
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Justin Early CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-08 15:08 EST by Vladimir Varankin CLA
Modified: 2017-04-11 15:12 EDT (History)
1 user (show)

See Also:

Attachments
Add tab size text field. @repo: org.eclipse.vjet.dltk (8.74 KB, patch)
2013-01-26 08:14 EST, Jack Liu CLA 		no flags Details | Diff
spaces are now used for tab (8.74 KB, image/png)
2013-01-26 11:30 EST, Justin Early CLA 		no flags Details
screenshot of preferences set to space (79.90 KB, image/png)
2013-01-26 11:31 EST, Justin Early CLA 		no flags Details
Indention is still could not be set (668.41 KB, image/png)
2013-03-21 18:39 EDT, Vladimir Varankin CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir Varankin CLA 2013-01-08 15:08:27 EST 
It's seems that setting indentation character in Eclipse→VJET→Editor preferences panel takes no effect. So tab char is always used.

Using latest stable Juno + "Eclipse VJET JavaScript IDE (Incubation)", 0.9.0.201301072009

BTW: Restarting IDE doesn't helps :)
Comment 1 Justin Early CLA 2013-01-08 15:15:02 EST 
confirmed was able to reproduce this issue
Comment 2 Justin Early CLA 2013-01-23 19:46:06 EST 
Control + Shift + F now supported. the auto indent still needs some work however to follow the format preferences.
Comment 3 Jack Liu CLA 2013-01-26 04:59:12 EST 
fixed.

One special case:
character : Tabs
tab size: 10
Indention size: 15

result: <Tab><space><space><space><space><space>
Comment 4 Jack Liu CLA 2013-01-26 08:14:19 EST 
Created attachment 226154 [details]
Add tab size text field. @repo: org.eclipse.vjet.dltk

The repo is:
org.eclipse.vjet.dltk
Comment 5 Justin Early CLA 2013-01-26 11:28:02 EST 
This patch works.

a5097a6227187b61b9c7184cd59a8bda86e39312
Comment 6 Justin Early CLA 2013-01-26 11:30:18 EST 
Created attachment 226155 [details]
spaces are now used for tab
Comment 7 Justin Early CLA 2013-01-26 11:31:24 EST 
Created attachment 226156 [details]
screenshot of preferences set to space
Comment 8 Vladimir Varankin CLA 2013-03-21 18:39:37 EDT 
Created attachment 228895 [details]
Indention is still could not be set
Comment 9 Vladimir Varankin CLA 2013-03-21 18:40:14 EDT 
Space indention still isn't work for me. See screenshot below

"Eclipse VJET JavaScript IDE (Incubation)", 0.10.0.201303212048 (Juno 4.2.2)
Comment 10 Justin Early CLA 2013-03-21 18:57:51 EDT 
Hi Vladimir,

Thanks for upgrading. 
Are you using an existing workspace (<0.10) or new workspace created with 0.10 upgrade?

Thanks,

Justin
Comment 11 Vladimir Varankin CLA 2013-03-22 02:36:05 EDT 
> Are you using an existing workspace (<0.10) or new workspace created with 0.10 upgrade?

Just checked with new clean workspace – same problem.