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 :)
Bye, Rouslan
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.
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 git and rebased my remote "indexes" branch on master.
Bye, Rouslan
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.
Bye, Rouslan
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.
Bye, Rouslan
Great; please look at this screenshot for a weird display in IE 8 / Windows XP.
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?
Rouslan
2011/11/15 Rouslan Placella rouslan@placella.com:
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?
I'll try to reproduce this tomorrow or Thursday. I have a similar setup on my laptop at work.
Kind regards,
Dieter
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.
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
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.
-
Dieter Adriaenssens -
Marc Delisle -
Rouslan Placella