ElkArte Community

Elk Development => Bug Reports => Exterminated Bugs => Topic started by: scripple on April 02, 2014, 01:08:09 am

Title: The sentence doesn't fit the crime
Post by: scripple on April 02, 2014, 01:08:09 am
Quote from: Elkarte Post PageThe message has the following error or errors that must be corrected before continuing:

Warning - while you were reading 3 new replies have been posted. You may wish to review your post.
I think there needs to be a way to relax that warning level a bit.  A full error box in the scary you've done something bad colors saying YOU MUST CORRECT this when all that's happened is someone posted while you were seems a bit extreme.

Title: Re: The sentence doesn't fit the crime
Post by: Antechinus on April 02, 2014, 01:35:16 am
 :D Come to think of it, that's funny. How is anyone supposed to "correct" it? Tell the miscreants to remove their posts immediately? Inform them that if, in future, they dare to post while you're composing something, they'll be shot at dawn?
Title: Re: The sentence doesn't fit the crime
Post by: scripple on April 02, 2014, 01:45:08 am
Yeah the top sentence really needs to go.  And I think there's a warning box rather than an error box that should probably be used here.
Title: Re: The sentence doesn't fit the crime
Post by: emanuele on April 04, 2014, 11:06:52 am
mmm.... yeah, sounds like ti should be some more soft...
Suggestions are welcome, I'll try to see how to "inject" the stuff into the code. ;D

TBH, that is one of the things to do with the posting interface: the "facilities" are all there (the errors class can handle different levels of errors), it just need to be "tweaked" (I think there is a topic in "feature discussion", I should find it...).
Title: Re: The sentence doesn't fit the crime
Post by: scripple on April 04, 2014, 11:17:34 am
Yeah if it can handle a warning box instead of an error box that would be a good start.   Otherwise I'm going to have to recolor my error boxes.   :P

As to new text really the first line needs to go and the whole message be the second one.
Title: Re: The sentence doesn't fit the crime
Post by: emanuele on April 04, 2014, 03:01:11 pm
I'm not sure it can just go, I have to check the code (improve that part is in the plans for 1.1, at least in my mind, not sure if I added to the main issue at github).
The color should be possible (just a matter of changing the error level... I hope.
Title: Re: The sentence doesn't fit the crime
Post by: emanuele on April 06, 2014, 01:39:43 pm
Here it is something:
https://github.com/emanuele45/Dialogo/commit/f276193ca5269ad05cc52cb85020ea88609d3378

I didn't dare removing the message entirely because it would be a bit of a pain (even though it may work), just made it less aggressive and generic.