[Phpmyadmin-devel] best approach about jQuery dialogs' title bar?

[Tyron Madlener]

I was just about to check again how it currently looks but the master/
demo seems broken. If I press 'Create table' from the navigation frame
in any database the whole content frame turns white after loading. If
I press 'create table' again, following error turns up in the console:
"Uncaught TypeError: Cannot read property 'scrollWidth' of null" -
this happens in chrome 14 / win xp.

Also, this Icon is broken (unsharp 200% zoom):
http://demo.phpmyadmin.net/master/themes/pmahomme/img/error.ico

On Wed, Oct 19, 2011 at 12:26 PM, Marc Delisle <marc at infomarc.info> wrote:
> Hi,
>
> Tyron Madlener suggested to get rid of the title bar in the create table
> dialog.
>
> I've come up with this patch (done here just for pmahomme):
>
> diff --git a/js/functions.js b/js/functions.js
> index 19ab7ce..d171496 100644
> --- a/js/functions.js
> +++ b/js/functions.js
> @@ -1514,7 +1514,6 @@ function PMA_createTableDialog( $div, url , target)
>               $div
>               .append(data.error)
>               .dialog({
> -                 title: PMA_messages['strCreateTable'],
>                   height: 230,
>                   width: 900,
>                   open: PMA_verifyColumnsProperties,
> @@ -1528,7 +1527,7 @@ function PMA_createTableDialog( $div, url , target)
>               $div
>               .append(data)
>               .dialog({
> -                 title: PMA_messages['strCreateTable'],
> +                 dialogClass: 'create-table',
>                   resizable: false,
>                   draggable: false,
>                   modal: true,
> diff --git a/js/messages.php b/js/messages.php
> index 96f759b..71afc05 100644
> --- a/js/messages.php
> +++ b/js/messages.php
> @@ -222,7 +222,6 @@ $js_messages['strReloadDatabase'] = __('Reload
> Database');
>  $js_messages['strCopyingDatabase'] = __('Copying Database');
>  $js_messages['strChangingCharset'] = __('Changing Charset');
>  $js_messages['strTableMustHaveAtleastOneColumn'] = __('Table must have
> at least one column');
> -$js_messages['strCreateTable'] = __('Create Table');
>  $js_messages['strYes'] = __('Yes');
>  $js_messages['strNo'] = __('No');
>
> diff --git a/themes/pmahomme/jquery/jquery-ui-1.8.16.custom.css
> b/themes/pmahomme/jquery/jquery-ui-1.8.16.custom.css
> index a712408..a087015 100644
> --- a/themes/pmahomme/jquery/jquery-ui-1.8.16.custom.css
> +++ b/themes/pmahomme/jquery/jquery-ui-1.8.16.custom.css
> @@ -437,6 +437,7 @@ button.ui-button::-moz-focus-inner { border: 0;
> padding: 0; } /* reset extra pad
>   */
>  .ui-dialog { position: absolute; padding: .2em; width: 300px;
> overflow: hidden; }
>  .ui-dialog .ui-dialog-titlebar { padding: .4em 1em; position: relative;  }
> +div.create-table .ui-dialog-titlebar {display: none; }
>  .ui-dialog .ui-dialog-title { float: left; margin: .1em 16px .1em 0; }
>  .ui-dialog .ui-dialog-titlebar-close { position: absolute; right:
> .3em; top: 50%; width: 19px; margin: -10px 0 0 0; padding: 1px; height:
> 18px; }
>  .ui-dialog .ui-dialog-titlebar-close span { display: block; margin: 1px; }
>
>
>
> Questions:
>
> 1. What do you think of this patch?
>
> 2. Should we instead remove the title bar for all our jQuery dialogs?
>
> P.S. We'll need to be extra careful when updating the jquery ui, by
> reinserting phpMyAdmin's customizations under themes.
>
> --
> Marc Delisle
> http://infomarc.info
>
> ------------------------------------------------------------------------------
> All the data continuously generated in your IT infrastructure contains a
> definitive record of customers, application performance, security
> threats, fraudulent activity and more. Splunk takes this data and makes
> sense of it. Business sense. IT sense. Common sense.
> http://p.sf.net/sfu/splunk-d2d-oct
> _______________________________________________
> Phpmyadmin-devel mailing list
> Phpmyadmin-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel
>