Bug 530627 - QuickFix for no-mixed-spaces-and-tabs needs improvement
Summary: QuickFix for no-mixed-spaces-and-tabs needs improvement
Status: NEW
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: JS Tools (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2018-02-01 14:45 EST by Carolyn MacLeod CLA
Modified: 2018-02-01 14:45 EST (History)
0 users

See Also:

Attachments
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 Carolyn MacLeod CLA 2018-02-01 14:45:07 EST 
- In Javascript Validation settings, change "Mixed spaces and tabs" to Error.
- Type a mix of spaces and tabs at the beginning of a line of code.
- You should see the red X.
- Hover over the red X to see what the suggested QuickFix is - it says it can disable the no-mixed-spaces-and-tabs error.

This is not a useful quickfix when you are trying to make a contribution to someone else's code.

It would be more useful to offer the user a choice of several possible QuickFixes:
- change beginning-of-line tabs to spaces
- change beginning-of-line spaces to tabs
- IF the project's no-mixed-spaces-and-tabs setting is "smart-tabs" then offer to change beginning-of-line whitespace to smart-tabs

Note that all 3 of these can have a Fix All option.

(In case it's useful, the initial implementation of no-mixed-spaces-and-tabs was added in Bug 462980)