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

Priority Can Include Characters Description Does Not State It Can Include #24

Open
sydneyfalk opened this issue Oct 10, 2018 · 2 comments

Comments

@sydneyfalk
Copy link

I think I may have found either a bug or a missed opportunity for data validation.

Creating a task with (A-) as the priority seemed to work -- it's stored that way, it's prioritized just under 'A', etc.

However, the stated requirements are one English alphabet letter, A-Z, uppercase only. So this probably is unintentional.

(Having said that, when I did see that (A-) worked, I was pleased as punch about the idea we could divide things more finely that way -- however, A+ comes 'after' A. So I figure it's probably unintentional behavior. It'd be neat if it did do 'plus and minus', but I understand that might run counter to the minimalist nature of the tool, too. So not requesting that by any means, but if it's interesting enough to anyone to run with, I wouldn't be unhappy about it.)

(Also, I'm not 100% sure if this should be under the CLI. I haven't tested it with any other interfaces, but I have no idea if they're doing validation above and beyond or what. Sorry if this is in the wrong place!)

@devingfx
Copy link

I think right now the +/- sign is ordered alphabetically (without math meaning)...

Btw, I'm not against the idea, but having already 26 level of priority isn't enough? Do you really needs 26 * 3 = 78 level of priority?

@loziniak
Copy link

having already 26 level of priority isn't enough

Can be troublesome if you want to put one task just after all (A)s but before (B)s. Then you have to change all Bs to C, all Cs to D etc.

It'd be neat if it did do 'plus and minus'

Sorting is done probably using ASCII codes (at least in Sublime Text when I press [F9], as I use it), so maybe instead of + you can use !, as it has code 33, which is sorted before )'s code 41?

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

No branches or pull requests

3 participants