From: Marc Delisle marc@infomarc.info
Aris Feryanto a écrit :
Hi Marc,
From: Marc Delisle marc@infomarc.info
Aris Feryanto a écrit :
Hi Marc,
From: Marc Delisle marc@infomarc.info Aris, I would
like to
merge this feature but I get merge conflicts, can you have a look at this?
I just fixed the merge conflict in my master branch. Kindly merge the grid editing from there. The master branch also contain some fix for recent tables and configuration for qTip hint. Thanks.
Done.
Btw, I'm currently refactoring and cleaning code in js/makegrid.js and there will be a significant code structure changes in that file. I'll update you once it is finished.
OK.
I finished cleaning code in js/makegrid.js and pushed the changes to my git repo.
I change the structure of the code to match existing PMA code, but I still keep the OOP style inside the function. The new PMA_makegrid() function can accept some optional parameters, in order to adjust which functionality (resize, reorder, show/hide, grid edit) that want to be enabled for a table. This may be useful for applying some features for other types of table (other than the one generated while browsing database).
-- Aris Feryanto
Pushed, thanks.
I am making a last attempt about this issue: do you have an idea about advertising (for example with a tip) about grid editing?
Oh, right. Adding tooltips in every editable cells will be disturbing, IMO.
How about add a hint tooltip in "Edit" link in table's rows? Maybe some hint like: "You can also edit a field by clicking directly on a cell in the table.". Actually, we may add more hint to tell the users that grid editing doesn't work for BLOB data type, but the hint will be too long to be displayed.
-- Aris Feryanto