search

nicholasmr / obblm

Automatically exported from code.google.com/p/obblm
26 stars 54 forks source link

Rewrite the "Success" and "Failure" output #72

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
There is no need for the Sucess and Failure messages to be output in a
table. I suggest showing something like the attached image, and then have
the message fade after x seconds.

Unless anyone objects, I'll make the required changes :)

Original issue reported on code.google.com by syko...@gmail.com on 13 Apr 2009 at 2:36

Attachments:

GoogleCodeExporter commented 9 years ago 
Fine with me. But give it at least 10 secs fadeout!

Original comment by Nimda...@gmail.com on 13 Apr 2009 at 3:31

GoogleCodeExporter commented 9 years ago 
Committed to trunk. I havent added the fadeout yet, will do that when jQuery is 
added
to the trunk.

You want it added for the next release, eventhough this is the only place it is 
used?

Original comment by syko...@gmail.com on 13 Apr 2009 at 8:57

GoogleCodeExporter commented 9 years ago 
No. I'd prefer leaving jQuery out until the following release after 0.38.

Original comment by Nimda...@gmail.com on 13 Apr 2009 at 11:11

GoogleCodeExporter commented 9 years ago 
...that is, what you have done now is fine for 0.38.

Original comment by Nimda...@gmail.com on 13 Apr 2009 at 11:12

GoogleCodeExporter commented 9 years ago 
I actually think I'd prefer if the message persists?

Original comment by Nimda...@gmail.com on 14 Apr 2009 at 6:53

GoogleCodeExporter commented 9 years ago 
Fine by me :)

Original comment by syko...@gmail.com on 14 Apr 2009 at 8:09

GoogleCodeExporter commented 9 years ago

Original comment by Nimda...@gmail.com on 14 Apr 2009 at 9:03

GoogleCodeExporter commented 9 years ago 
There is a problem. It seems that when the status bar shows up it removes the 
padding
space at the very bottom of the page (generated by index.php at the end of the 
file
as a div with clear: both).

Original comment by Nimda...@gmail.com on 15 Apr 2009 at 12:34

GoogleCodeExporter commented 9 years ago 
Whops, there was an unneeded closing div tag in there, works now, fix comitted.

Original comment by syko...@gmail.com on 15 Apr 2009 at 12:45

GoogleCodeExporter commented 9 years ago 
Cheers.

Original comment by Nimda...@gmail.com on 15 Apr 2009 at 12:46