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
base: develop
Are you sure you want to change the base?
Conversation
package.json
Outdated
}, | ||
{ | ||
"command": "org.updateSummary", | ||
"key": "ctrl+alt+o #", |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
In {
"command": "org.updateSummary",
"title": "Org: Update Summary"
},
{
"command": "org.toggleCheckbox",
"title": "Org: Toggle Checkbox"
} |
news ? |
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; |
There was a problem hiding this comment.
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).
There was a problem hiding this 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.
Implements nested checkbox lists and summaries (number of checkboxes ticked relative to total number of checkboxes on the same level as ratio or percentage).