![]() ![]() |
Still broken. |
![]() ![]() |
Well, now that Jesse seems to be back, can we get this issue FINALLY fixed? Pretty please? |
![]() ![]() |
Will work on it tomorrow - http://userscripts.uservoice.com/forums/795-gen... |
![]() ![]() |
One month later - is it tomorrow yet? :P |
![]() ![]() |
Still broken... |
![]() ![]() |
I definitely miss this feature. :`-( +1 love and support for Jesse Andrews |
![]() ![]() |
Fixed. Was a combination of image resize code and cloud storage configuration both being messed up :( |
![]() ![]() |
I seem to be getting a display issue when viewing the screenshot. As soon as I click on a screenshot with a caption (specifically, the one on this script), I see the image fade in followed a second later by the caption. The caption seems to render on the left side of the page, then after a half-second it snaps to the center (where it should be). Another issue is an outline on the actual image upload portion. See this screenshot:
Also, are you planning on adding an option to SORT the images. That was major complaint that some people had. Currently, in order to get them in the right order, you have to re-upload all screenshots in the correct order. Mozilla/5.0 (Windows NT 5.1; rv:5.0) Gecko/20100101 Firefox/5.0 |
![]() ![]() |
Cletus wrote:Buy a faster CPU/GPU? ;) hehe... that is how the rails version of lightbox has worked here in the past. Many thanks for reenabling Sir Jesse. :) |
![]() ![]() |
Marti wrote:It is still an issue though, and a tacky one at that. |
![]() ![]() |
Cletus wrote:Looks fine to me. Your current solutions are don't use a caption, go bug lightbox, go bug rails, buy a faster computer/video card. :P ;) :) It may help to have a newer jQuery, if not present, though as I'm sure it may be better optimized *hopefully* |
![]() ![]() |
Cletus / Marti, If someone with firebug/web-inspector wants to recommend CSS rules to fix it, I'm all :) |
![]() ![]() |
Jesse Andrews wrote:This script attempts to fix some CSS defaults including the borders... it is really short and simple so should be okay to import at least the border fix on screenshot uploads I think although I haven't tried it cross-browser. As far as lightbox (facebox as named by USO) goes I would have to relook at the jQuery plugin and see what jQuery/plugin is doing to make the caption "slide" into the picture... it's possible that it's just an artifact of the old jQuery animation engine that is in the current all.js. I made an attempt to rewrite the lightbox routine client side but found a cross-domain restriction in jQuery which pretty much killed everything and it was like so two weeks ago. ;) Jesse Andrews wrote:In general I believe the last time I looked at the site CSS it could use some TLC to minimize it's footprint. (e.g. when setting color: #222222; in screen.css you could drop off some of those twos... color: #222; is the same thing. Same with a:visited { color: #000066; } would be a:visited { color: #006; } Much as I like static color names too... most color names are longer than their 4 to 7 digit hex codes. top: 0px; could be shortened to top: 0;, etc.).
Just a note too IE10 is supposed to be dropping conditional comments for CSS. |
![]() ![]() |
Jesse Andrews wrote:Fix for lightbox issue I described: screen.css, line 985 FIND #jquery-lightbox { position: absolute; top: 0; left: 0; width: 100%; z-index: 100; text-align: center; line-height: 0; }
ADD AFTER #jquery-lightbox > .ui-effects-wrapper {margin: 0 auto !important;}
The issue with this one is that the caption was getting wrapped in another div (.ui-effects-wrapper) during transition which had an element.style with a margin: 0;.
Fix for border issue I described:
#facebox td { border-bottom: 0; padding: 0; }
CHANGE TO #facebox td { border: 0; padding: 0; }
|
![]() ![]() |
Cletus wrote:This makes the caption slide down, stall occasionally and flash on slower computers instead of sliding left to right quickly... not any better and appears to be worse for the wear imo. |
![]() ![]() |
i tried to upload an image (named titlePNG.PNG), but it failed. Then i renamed it to titel.PNG (German translation for title) and then it works. Anyhow hope i could help you people. (Good feature :) ) |
|
|
Screenshot upload works only after several tries. |
![]() ![]() |
Not sure if anyone tried that, but to delete the actual icon you just need to click in "Upload a new icon" and then upload nothing. |
![]() ![]() |
Could this be changed in such a way that it will auto-default to the icon supplied by the script itself?
That means it should be able to handle things like "@icon data:image/png;base64,xxxxxxxxx"
|
![]() ![]() |
©TriMoon™ wrote:See this script. |
![]() ![]() |
@Marti,
FYI: Just to be 100% sure you know what i mean:
|
![]() ![]() |
©TriMoon™ wrote:I did... although the direction of taking it out of the @icon into the browser wasn't fully anticipated. Provided it was always a data:URI this could be feasible in a user.js file that you could make... however there would be a serious concern for regular browsing the site privacy if not which would make this idea as default a -1. It's better just to upload the script icon to USO (yes I know it takes multiple attempts and fails if no suitable upload bandwidth is available...just be patient and try, try again :) and reference that url in your script just like the script aforementioned allows... this also keeps the scripts of reasonable size since data URIs can be really big!. USO really shouldn't become a CDN for script icons through user.js. I appreciate the details on the idea and perhaps it will be discussed further. :) |
![]() ![]() |
still broke ? because i cant upload any screens anymore... :/ Regards
|
![]() ![]() |
From my experience and currently in non-secure USO access...
Keep trying and it should eventually do it... I've done all kinds of resolutions too so the report of images being too small doesn't appear to be correct. Mozilla/5.0 (X11; Linux x86_64; rv:7.0.1) Gecko/20100101 Firefox/7.0.1 Mozilla/5.0 (X11; Linux x86_64; rv:11.0) Gecko/20100101 Firefox/11.0 Mozilla/5.0 (X11; Linux x86_64; rv:18.0) Gecko/20100101 Firefox/18.0 |
![]() ![]() |
i wish animated gifs were supported |






