610799 - Add character counter to idea submission page

Status: VERIFIED FIXED

(Input :: General, enhancement, P1)

Description

[Fred Wenzel [:wenzel]]

Once idea submissions land, we need to add the same character counter to it that we have for feedback now.

Refer to bug 573817 for the character limits and colors that should be applied.

Comment 1

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Aakash - do you want the same character count limit for happy/sad submissions to be applied to Ideas?  Or do you want a different limit?  Fwiw, the happy/sad pages are using an input text field, while the suggestion submission page is using a textarea.

Comment 2

[Aakash Desai [:aakashd]]

Nope, it'll need to be 250, 50 (orange), 25 (red), 0, -1, etc.

Comment 3

[Aakash Desai [:aakashd]]

I think this is done, right Ryan?

Comment 4

[Aakash Desai [:aakashd]]

Never mind, needed to read comment #2

Comment 5

[Ryan Snyder [:ryansnyder] [:rsnyder] [:rysny]]

Okay, this now counts down for 250 characters for suggestions and 140 characters for praise and issues.  

I updated the JS a bit (to reduce code) so that the happy/sad/suggestion pages all turn orange then they reach 20% of the max characters, and red at 10% of the max characters.  It will change colors on the thresholds listed above, but will be slight different on praise / issues pages (will change at 36 characters and 18 characters).

Committed.

==

https://github.com/fwenzel/reporter/commit/8bb543

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; we also filed bug 616230.

* http://input.stage.mozilla.com/en-US/suggestion has 250 chars
* http://input.stage.mozilla.com/en-US/happy has 140 chars
* http://input.stage.mozilla.com/en-US/sad has 140 chars

Comment 7

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Added to litmus 15029

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #7)
> Added to litmus 15029

Then you want the In-Litmus+ flag, not In-Testsuite+; adjusting.

Comment 9

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Ah, sorry. I am working on an automated test but it's not complete yet.

Comment 10

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Selenium test now in main test repo.