[vlc-devel] [PATCH 2/2] new LUA HTTP Interface Part 2 of 2

[Kaarlo Räihä]

2011/6/16 Francois Cartegnie <fcvlcdev at free.fr>

> Le jeudi 16 juin 2011, vous avez écrit :
> > This is part 2 of the new LUA HTTP Interface. I apologize for the size of
> > these patches, but since this is a complete replacement of the existing
> > HTTP interface, it is somewhat unavoidable. This patch removes the
> > existing interface and replaces it with a cleaner more user-friendly
> > interface.
>
> Testing your patches.
>
> You could have included the minified JQuery files: See my previous message
> to
> avoid git complaining about large lines.
>

IMHO it is better to have non-minified Javascript when they are shared like
this
- Installers are compressed (so the size difference shouldn't be big)
- It is easier to debug
- Possible patches are easier for eye


>
> Here's few things that will need to be fixed:
>
> - Not validating XHTML. A min of 23 errors on main page.
> - Units are missing on Offsets dialog
> - need to apply a vertical-align:top on tds (create stream dialog)
> - set table width to 100% (create stream)
> - 'Open' Stream Button -> 'Create' Stream (???)
> - Media Browser needs to start with a height (in case of large number of
> dirs)
> - Media Browser needs clickable links of the same color as other dialogs
> - Media Browser shouldn't need a double click, or if it allows multiple
> selection, must toggle the style of selected items
> - Audio slider is not really working
> - You should not use the same color as Title Bars for SlideBars.
> - Playlist should only show filenames (display full path as tooltip)
>
> What's the test results with other browsers ?
>
> Francois
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110617/ab9c4f7d/attachment.html>