On 13 Jun 2011, at 20:31, Rouslan Placella <rouslan@placella.com> wrote:
On Mon, 2011-06-13 at 08:09 -0400, Marc Delisle wrote:
Rouslan Placella a écrit :
On Sat, 2011-06-11 at 21:23 -0700, Aris Feryanto wrote:
Hi Rouslan,
----- Original Message -----
From: Rouslan Placella <rouslan@placella.com>
On Sat, 2011-06-11 at 10:58 +0700, Aris Feryanto wrote:
On 10 Jun 2011, at 20:31, Marc Delisle <marc@infomarc.info> wrote:
I've updated my repo and it should be available in the demo soon. I add
small popup balloon when we hover the column header. I also add some css code to
prettify the column dragging. Please kindly check this.
P.S.: All my changes for column resize and reorder can be found under
"colresize" branch in my git repo.
I found a small bug with the tooltip. See video here:
http://www.placella.com/pma/colreorder_bug.avi
Thank you for reporting this. It is nice to show that in the video. I tested in some browsers (chrome, firefox, IE) and found that the bug only occurred in firefox/ubuntu. I've fixed this in my git repo.
It works great now. By the way, I found another few small issues that
you might want to look at :)
You hard-coded a lot of sizes in CSS in px units. So now items don't
really scale. For example, go to main PMA page and from the dropdown
select font size of, for example, 132%. Now go to browse a table and
hover the column header. You will see your "Drag to reorder" hint is the
same as when the font size was set to 82%, but instead it should be
bigger. You can use the 'em' unit instead of the 'px' unit in CSS to fix
this.
Also, while resizing a column, the "Drag to reorder" hint keeps trying
to pop out when you hover the label of the column. And when you resize
the column to a small size, about 10px, and then hold the mouse still a
bit both the hints for sorting and reordering pop out (see attached
screenshot). And at this point I think that "Sort" alone doesn't mean
much any more, so in my opinion the two hints should be merged into one
like "Drag to Reorder <br /> Click to Sort".
Just one hint could be confusing because there are many possibilities:
- click on link to sort
- click in the <th> (but not on the link) to mark/unmark the column
- drag the <th>
Looks like Aris already picked up on my suggestion about the combined
hint. I think it works quite nice in that it only shows "Click to sort"
when you hover the link. I guess that at this point maybe the hint could
contain the same way "Click to mark" when one is over a <th>, but not
the link.
Hi,
Just realized my previous email seem not delivered successfully to phpMyAdmin-devel list. I've previously sent an email telling that I will work on merging the tooltip as suggested by Rouslan. I'm still working and testing on this and will add the 'Click to mark'.
I am also fixing the previous code, as it hasn't checked whether we are under Browse tab, before calling the column-order remembering function.
I wonder if there is any existing exact way to check if we are under Browse tab? Current code do the checking using requested url pattern and analyzed SQL query (you may see function PMA_isBrowsing in file libraries/display_tbl.lib.php in my git repo). But, this is not enough for handling load from AJAX request under other tabs (e.g. AJAX request to sql.php caused by pagination of results under SQL tab)
--
Aris Feryanto