-
Notifications
You must be signed in to change notification settings - Fork 8
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
Don't limit Topic title to 100 chars #2
Comments
Would still like this implemented. :) |
@moll Added to the project with a promising name. The thing is, the wish-list will go through review an prioritization at some point upcoming weeks we'll prioritize these and start crunching on the list. You can always pull request a fix if in a pinch. This issue MAY require some work in Citizen OS FE. More - if you have some issues that have annoyed you for extended period of time, please add them to Winter Push For Perfection (WPFP) project. I cannot guarantee delivery in that project, but I can guarantee that we will go through them with the project, prioritize them and give feedback on why and where they stand in the list of work. |
ACK on fixing it myself. I'll ask @infokujur once we've got our instance set up. Btw, I don't have permissions to move issues to projects. |
@moll OK, I'll review the project permissions and let's fix. Discussion move to Slack. |
@moll Remind me what was your proposal on this?
|
For all practical purposes, remove the limit. For practical purposes, bump it up to either half or a full thousand to prevent it accidentally including the body entirely. |
@moll We, me and @loorm, had a discussion on the limit. We would like to keep some kind of a limit. What are the practical issues you're having because of the limit? |
Well, first, that limit has to then be enforced in Etherpad. Hacking off someone's title after signaling that it's perfectly okay to enter it is not a good way to enforce said limit. Secondly, limiting entry and limiting display should not be conflated. There are, albeit rudimentary, ways in CSS to truncate text. E-mails differ in length limits compared to a 3-line square on the home page and may need further truncating anyways. Rahvaalgatus already has a title limit in the initiative wizard, but Etherpad permits overflowing that anyways. I'm arguing that the backend needs to get out of the truncation game and either start limiting OR let the frontend decide how to handle longer titles. |
It's been a while, this is what we decided to do after discussion with @ilmartyrk and @loorm:
|
Parse and store whole title, let client decide what to do.
Related tasks:
TODO
The text was updated successfully, but these errors were encountered: