[Phpmyadmin-devel] Improved index editor

[Marc Delisle]

Rouslan Placella a écrit :
> On Wed, 2011-11-16 at 06:30 -0500, Marc Delisle wrote:
>> Le 2011-11-15 16:42, Rouslan Placella a écrit :
>>> On Tue, 2011-11-15 at 15:20 -0500, Marc Delisle wrote:
>>>> Rouslan Placella a écrit :
>>>>> On Tue, 2011-11-15 at 12:22 -0500, Marc Delisle wrote:
>>>>>> Rouslan Placella a écrit :
>>>>>>> Hi all,
>>>>>>>
>>>>>>> I did some work on the "table properties" page and, in particular, on
>>>>>>> the index editor. As a result, I now have a branch with several
>>>>>>> improvements and fixes. I would like to merge with master within the
>>>>>>> next few days, unless there are some objections.
>>>>>>>
>>>>>>> Fixes:
>>>>>>> * Fix for indexes list that did not respect $cfg['InitialSlidersState']
>>>>>>> * Removed some redundant data from AJAX replies
>>>>>>> * "Add columns" action only worked when the English language was used
>>>>>>>
>>>>>>> Features:
>>>>>>> * AJAXified the "create index" action (not sure why this wasn't done)
>>>>>>> * Tidied up the layout at the bottom of the table structure page with
>>>>>>> some fieldsets
>>>>>>>
>>>>>>> * Index editor:
>>>>>>> Some issues with the old implementation of the index editor are:
>>>>>>> It was using up way too much horizontal space (900px), by tidying up the
>>>>>>> layout of the editor, I brought this down to 450px. I converted the
>>>>>>> "primary key warning" into a tooltip to gain a bit of vertical space.
>>>>>>> Then there was an issue when a user requested to add a column to the
>>>>>>> index editor: the browser fired an HTTP request of the form to the
>>>>>>> server, just to get the same form back with an extra empty field. Of
>>>>>>> course, this action can be done on the client side by just cloning a row
>>>>>>> from the table.
>>>>>>>
>>>>>>> Anyway, there are some low-quality screenshots attached. You can also
>>>>>>> pull from the "indexes" branch of
>>>>>>> git://repo.or.cz/phpmyadmin/roccivic.git
>>>>>>>
>>>>>>> And, your feedback is very welcome :)
>>>>>> Hi Rouslan,
>>>>>> this is a welcome improvement to the structure page and index editor. 
>>>>>> However there is one case that needs handling. If the creation of a 
>>>>>> unique index generates a duplicate key error, the user is not warned and 
>>>>>> the "Add index" panel remains there.
>>>>> That was actually an unrelated bug, introduced with the sticky menu bar.
>>>>> I fixed that in master.
>>>> Great; please look at this screenshot for a weird display in IE 8 / 
>>>> Windows XP.
>>> I can't reproduce this issue in IE 8 / Windows 7. Could you look into
>>> it? Maybe it's the browser cache?
>> At home I have IE 9 / Windows Vista. I can reproduce the problem, even
>> after clearing the browser cache.
>>
>> More info: if I choose index type PRIMARY then hover the mouse over the
>> field to choose a column name, suddenly the index name and type fields
>> enlarge.
> 
> I still can't reproduce the issue on any machine I've tried so far...
> 
> Could it be a git issue? I rebased my indexes branch several times and
> I'm not sure how well git works with pulling from rebased branches.
> Could you try and delete my branch from your local repo and clone it
> again?
> 
> Rouslan

I did that and still have the problem. Well, you can merge the feature, 
then we'll have more testers and more feedback.


-- 
Marc Delisle
http://infomarc.info