jgraglia / Trello-Points

a fork of TrelloScrum
28 stars 6 forks source link

points in parenthesis interfere with custom info in parenthesis #6

Open slav opened 12 years ago

slav commented 12 years ago

I sometimes put small additional information inside regular parenthesis inside card title. That confuses Trello Points and interferes with it. Furthermore if I click on points button to add points it replaces my custom text.

Just a suggestion, but store the points in the title inside square [ ] or curly { } parenthesis. If you're concerned about usage and compatibility with existing cards, perhaps make it an option?

Thank you!

jgraglia commented 12 years ago

Hi, I know there is a pb with the point detection system... I'll try to fix it soon.. Thx for your comment!

slav commented 12 years ago

Any luck changing parenthesis used?

jgraglia commented 12 years ago

Hi. My week-ends are too short ;-) I'll try to publish a release soon.. if I can found the valid regexp for parsing...

slav commented 12 years ago

How're you trying to parse it? I'll write regex for you.

jgraglia commented 12 years ago

Hi, I have 2 parsers : one for numerics and others with everything I have wrote some unit tests for that : test/alltests.html & test/parser-test.js (and parser.js) with differents testcases and expected result. What I want to have is "My Card (Hello) with the last paranthesis (HI)" have to return "HI" as the points "Another example with numeric (dddd and (?)" -> ? "another one (9) but ll (90)" -> 90

For now I'm using thoses regexp :

/* parse regexp- accepts digits, decimals and '?' / var positiveNumericRegexp=/((\x3f|\d.?\d+))$\s?/m; /* allow negative numbers : - / var signedNumericRegexp=/((.))$\s?/m;

/* allow all chars, non numeric points / var numericAndAlphaRegexp=/((.))$\s?/m;

AsaAyers commented 12 years ago

Is there a reason to ever return non-numeric points other than "?"? It seems like any text in parenthesis should be ignored.

slav commented 12 years ago

Well, letters are used for t-shirt sizing, s, m, l to indicate size of the task. Numbers tend to get people to think in terms of hours. S, m, l, etc. forces people to think about relative size.

p.s. Sorry haven't gotten to regex expression. Got swarmed with work.