> There are several other things that aren't right with the server status
> pages. In general I find that there are many problems related to the
> tabbed navigation, so I'm splitting the different server status features
> into separate pages. This should also improve further maintainability of
> the system.

Oh yes. That's probably a good idea.

> By the way, will you be hacking the server status pages any time soon?
> We might want to try and avoid any merge conflicts, if you do...

The only thing I might do is fix some of the bugs and glitches that I notice, which however should be only very local changes. If I do, I'll make it as a pull request on github. I'll leave a note here if I plan to make any bigger changes :-)


On Mon, Dec 3, 2012 at 11:45 PM, Rouslan Placella <rouslan@placella.com> wrote:
On 03/12/12 18:39, Tyron Madlener wrote:
> Sadly there also seems to be a big memory leak with jqplot. Letting the
> monitor run for a longer time causes my Chrome consume 1-2 GB of memory.
> Maybe old data values are not being deleted?
>
>
> On Mon, Dec 3, 2012 at 6:29 PM, Tyron Madlener <tyronx@gmail.com
> <mailto:tyronx@gmail.com>> wrote:
>
>     Heya. Haven't written in a while. This appears when I browse a table
>     from my forked github pma of today:
>
>     *Parse error*: syntax error, unexpected T_PAAMAYIM_NEKUDOTAYIM in
>     *E:\htdocs\nikolaus\pma\phpmyadmin\libraries\DisplayResults.class.php* on
>     line *3879*
>
>
>     The seems to be a lot of new bugs + glitchy behavior in the Status
>     monitor, since the transition to jqplot. Here are a few i could pin
>     down:*
>
>     - roccivic for some reason deletes the local storage when the page
>     unloads, though it's intention was for storing settings:
>     https://github.com/phpmyadmin/phpmyadmin/commit/9d4799e6728adf792dc955a59a868d61adad2be6#commitcomment-2243979

Yep, my bad. I'll put that back some time soon.

>     - Reloading the page while in the monitor will result in a js error
>     in jqpload and not load the charts
>
>     - The query analyzer dialog doesn't show the profiling chart
>     anymore. For this however I took the freedom to fix this + graphical
>     glitches myself and send a pull request on github:
>     https://github.com/phpmyadmin/phpmyadmin/pull/113

There are several other things that aren't right with the server status
pages. In general I find that there are many problems related to the
tabbed navigation, so I'm splitting the different server status features
into separate pages. This should also improve further maintainability of
the system.

By the way, will you be hacking the server status pages any time soon?
We might want to try and avoid any merge conflicts, if you do...

>     On another note:
>     - Mousewheel scrolling doesn't work in the left pane when I
>        1. Expand a DB with many tables
>        2. Expand the tables
>        3. Switch to page 2
>
>     - Also the left pane scroll bar is so massively fat, is that intended?

Yes, that's how I intended it to be. JM, had provided a thinner
replacement version, but that also came with a list of bugs that IMO
didn't justify the benefits.

>     Sorry for throwing these bug reports out so chaotically. Thanks for
>     reading anyway!


------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
_______________________________________________
Phpmyadmin-devel mailing list
Phpmyadmin-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel



--
Tyron Madlener
Anego Web Development

www.tyron.at
T: +43 650 5950596
E: office@tyron.at