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

Update for more counting statistics (i.e. sentences, paragraphs, etc.) #26

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

adrianinc
Copy link

Had a go at adding more functionality to the word counter.
Count Highlighted Words
Took a different approach to @jfgilmore by making the status bar item change to display the number of selected words rather than showing both stats together all the time. Thought it might be a nice alternative implementation since it reduces clutter on the status bar.

Additional Counting Features
On clicking the word count status bar item or typing the command "View Word Count Summary", a notification with counting data for the document is displayed including the count of character (without spaces), sentences and paragraphs.

Added a command to view a summary of the document count data for when the status bar item (word counter) is clicked.
Adds counting the selection and a command called when clicking the status bar item that displays the number of words, characters (without spaces) and sentences (basic implementation which just uses the number of full stops) either of the whole document or just the selection.
Counts paragraphs based on the definition that a paragraph is a sequence of characters separated by 1 more more blank lines.
Added the ability to count paragraphs to the feature listing
@msftclas
Copy link

msftclas commented May 17, 2020

CLA assistant check
All CLA requirements met.

c-harding
c-harding previously approved these changes May 19, 2020
Copy link

@c-harding c-harding left a comment

Choose a reason for hiding this comment

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

Nice!

Base automatically changed from master to main February 19, 2021 16:24
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.

4 participants