4 ms·
First of all, this is awesome. Great idea, can't believe it hasn't been around till now, and will definitely be using it. The only thing I could change here is
by jenius 13y ago
First of all, this is awesome. Great idea, can't believe it hasn't been around till now, and will definitely be using it.
The only thing I could change here is the "x" icon when expanded. Someone mentioned here that they would be unnerved hitting an "x" button with a lot of text - this is definitely valid, I felt the same thing when trying it out. A simple change to an "exit full screen" icon (like this http://findicons.com/files/icons/1667/iconic/32/fullscreen_exit.png http://findicons.com/files/icons/1667/iconic/32/fullscreen_e...), would solve the problem nicely : )
- afshinmeh 13y agoNice! Thanks for the nice suggestion.
- nfvs 13y agoAlso, the "expand" button overlaps with the h-scroll bar when it's displayed. http://i.imgur.com/4mwx7ty.png http://i.imgur.com/4mwx7ty.png Awesome project nevertheless.
- jamiecurle 13y agoThis is a very nice implementation. I did something similar myself django-writingfield[1], but it is Django specific. I much prefer your UI and the switching of dark text/light background is really slick. [1]: http://django-writingfield.obscuremetaphor.co.uk/ http://django-writingfield.obscuremetaphor.co.uk/
- StavrosK 13y agoI will second it. I would also move it on the bottom right, as I didn't look at it except with my peripheral vision and thought it was an X/close button. When instructions said to click it, I went from "why would I click the close button" to "oh, it's actually a fullscreen button" before I clicked it. I never would have discovered it.
- jjmardlin 13y ago+1 "X"s are frightening. You might just give it some tooltip text.
- davedx 13y agoIf I remember rightly, Github has something just like this for its pull request description textarea. :) It is a very nice UI control.
- tripzilch 13y agoGood point, I'd suggest using a green checkmark instead (which basically means "OK" instead of the X's "Cancel").