Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 318138 - Provide whitespace feedback
Summary: Provide whitespace feedback
Status: CLOSED FIXED
Alias: None
Product: Acceleo
Classification: Modeling
Component: Core (show other bugs)
Version: 3.0.0   Edit
Hardware: PC Windows Vista
: P3 enhancement
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2010-06-28 02:47 EDT by Ed Willink CLA
Modified: 2010-08-16 11:23 EDT (History)
1 user (show)

See Also:

Attachments
Patch v1.0 (22.40 KB, patch)
2010-08-12 04:06 EDT, Stephane Begaudeau CLA 		no flags Details | Diff
Screenshot of the result (38.30 KB, image/png)
2010-08-12 04:06 EDT, Stephane Begaudeau CLA 		no flags Details
Patch v2.0 (22.05 KB, patch)
2010-08-12 10:47 EDT, Stephane Begaudeau CLA 		no flags Details | Diff
Patch v3.0 (23.58 KB, patch)
2010-08-16 05:05 EDT, Stephane Begaudeau CLA 		laurent.goubet: iplog+
Details | Diff
Show Obsolete (2) 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 Ed Willink CLA 2010-06-28 02:47:33 EDT 
It is difficult to determine how new lines and whitespace contribute to the output.

Suggest:

Add new line and whitespace markers, and use a pale ? grey background for _every_ character that contributes to the text output.
Comment 1 Stephane Begaudeau CLA 2010-08-12 04:06:08 EDT 
Created attachment 176441 [details]
Patch v1.0

This patch will add a new action in the toolbar that will show all the whitespace that will be generated.
Comment 2 Stephane Begaudeau CLA 2010-08-12 04:06:27 EDT 
Created attachment 176442 [details]
Screenshot of the result
Comment 3 Ed Willink CLA 2010-08-12 04:19:01 EDT 
Screenshot looks great; hope it's in M1 or at least M2.
Comment 4 Stephane Begaudeau CLA 2010-08-12 10:47:20 EDT 
Created attachment 176475 [details]
Patch v2.0

New version of the patch with a better detection of the whitespace characters that need to be displayed. Raw version of the patch. It can have a significant impact on the performances of the editor when active.
Comment 5 Stephane Begaudeau CLA 2010-08-16 05:05:02 EDT 
Created attachment 176658 [details]
Patch v3.0

New version of the patch with improved performances.
Comment 6 Laurent Goubet CLA 2010-08-16 11:22:46 EDT 
This new version has been committed on HEAD and will be available in 3.1.0M1.