Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

clarify Preferences > Languages > Tab settings #483

Merged
merged 4 commits into from
Feb 24, 2023

Conversation

pryrt
Copy link
Contributor

@pryrt pryrt commented Feb 20, 2023

No description provided.

@pryrt
Copy link
Contributor Author

pryrt commented Feb 20, 2023

@alankilborn,

Does this phrasing work for you?

Copy link

@alankilborn alankilborn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"If the text is replacing Tabs with spaces..." seems awkward.

@alankilborn
Copy link

"If the text is replacing Tabs with spaces..." seems awkward.

Suggested change:

If Notepad++ is replacing Tabs with spaces, pressing the Tab key will insert enough literal spaces so that the next character will start at the beginning of the next tab stop.

@pryrt
Copy link
Contributor Author

pryrt commented Feb 21, 2023

The paragraph for the "tab size" seems to repeat basically the same information in the next two paragraphs, but more strangely worded.

I have reworked that section some more, and also embedded the image from the Editing > TAB-to-space/Space-to-TAB section.

@pryrt
Copy link
Contributor Author

pryrt commented Feb 21, 2023

Also, I will wait to see what happens with notepad-plus-plus/notepad-plus-plus#13186, in case that changes the wording needed here. ;-)

@pryrt
Copy link
Contributor Author

pryrt commented Feb 24, 2023

Since the "Replace by space" wording change wasn't included in the "adjacent" no-more-popup PR/commit, it may be a while before the wording gets changed (if ever). I don't want it waiting here indefinitely, so I'll do a separate fix to the manual if that changes in the future.

@pryrt pryrt merged commit 718b1b4 into notepad-plus-plus:master Feb 24, 2023
@pryrt pryrt deleted the tabToSpace branch February 24, 2023 18:57
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