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

Checkboxes and summaries #129

Open
wants to merge 30 commits into
base: develop
Choose a base branch
from

Conversation

aboukirev
Copy link

Implements nested checkbox lists and summaries (number of checkboxes ticked relative to total number of checkboxes on the same level as ratio or percentage).

sipi and others added 28 commits October 5, 2018 13:15
package.json Outdated
},
{
"command": "org.updateSummary",
"key": "ctrl+alt+o #",

Choose a reason for hiding this comment

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

I don't think "#" is a valid keybinding in vscode, but I figure most of the default keybindings we have aren't finalized anyhow

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for review. I replaced '#' with '/', which works. The '#' should have been Shift+3 but it's too many keypresses.
I also added command contribution entries with titles and descriptions.

@NateLehman
Copy link

NateLehman commented Oct 30, 2018

In package.json around line 100 titles should be added for the commands to be accessed the from the palette. Something like

{
    "command": "org.updateSummary",
    "title": "Org: Update Summary"
},
{
    "command": "org.toggleCheckbox",
    "title": "Org: Toggle Checkbox"
}

@WaYdotNET
Copy link

news ?

@mrkeuz
Copy link

mrkeuz commented Feb 11, 2021

Love checkboxes! Thanks! @aboukirev need rebase for fix conflicts.

PS: Can test it on my local machine.

// Percentage is a cookie indicating the percentage of ticked checkboxes in the child list relative to the total number of checkboxes in the list.
const percentRegex = /\[(\d*)%\]/;
const indentRegex = /^(\s*)\S/;
let orgTabSize: number = 4;
Copy link

@mrkeuz mrkeuz Feb 11, 2021

Choose a reason for hiding this comment

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

Is there a reason to store orgTabSize globally. It can be changed on fly. And potentially break traversing via trees. I think it potentially can have corner-cases here. If I understand your logic correctly (sorry if I'm wrong, just made fast review yet and not JS developer).

Copy link

@mrkeuz mrkeuz left a comment

Choose a reason for hiding this comment

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

Looks good for at fast review. Will tray to sync with current develop and test it editor.

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.

None yet

5 participants