Skip to content

[Feat] Have option to use tabs instead of spaces (forced override)#267

Open
jomillerOpen wants to merge 3 commits into
microsoft:mainfrom
jomillerOpen:main
Open

[Feat] Have option to use tabs instead of spaces (forced override)#267
jomillerOpen wants to merge 3 commits into
microsoft:mainfrom
jomillerOpen:main

Conversation

@jomillerOpen
Copy link
Copy Markdown

Read the default text editor preferences from Eclipse instead of defaulting to spaces by default. To avoid having additional plug-in dependencies, we use the existing preferences plug-in to check the multiple workspace instances for the settings. The 2 settings are the "spacesForTabs" and "tabWidth". If those settings are found, we will use those for the formatting options. If not, then it will continue to fall back to using defaults of using spaces. These settings are in effect when a project is neither a Java or C project type.

Read the default text editor preferences from Eclipse instead of defaulting to spaces by default.
Copilot AI review requested due to automatic review settings May 28, 2026 13:25
@jomillerOpen
Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree company="OpenText"

@jomillerOpen
Copy link
Copy Markdown
Author

Implements/satisfies #238

Switch to using the Platform.getPreferencesService() method instead, which will correctly account for the instances.
@jomillerOpen
Copy link
Copy Markdown
Author

I didn't know about that until now! But yes, I tried using it and it seems to work find as well. I'll switch it to that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants