View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0017584 | MMW 5 | General | public | 2021-02-20 01:13 | 2021-02-26 19:38 |
| Reporter | drakinite | Assigned To | |||
| Priority | urgent | Severity | tweak | Reproducibility | always |
| Status | closed | Resolution | reopened | ||
| Product Version | 5.0 | ||||
| Target Version | 5.0 | Fixed in Version | 5.0 | ||
| Summary | 0017584: Interface performance optimizations | ||||
| Description | To reduce browser layout recalculations during layout changes: 1) in splitter.js, removed the code that resets sibling minWidth/minHeight 2) moved analyzeSiblings call to the top of mouseDownHandler in splitter.js 3) Instead of changing document.body's class, its cursor style is set directly via JS, in splitter.js and control.js 4) Inside templates_add for Material Design(s) and Metro M, album art colors are cached in memory so that repeatedly-opened albums do not have to recalculate colors. 5) Changed delayedPopup delay from 400ms to 20ms in listview, to make the album grid open/close much faster. Tracked these changes in case of regressions. | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 2316 | ||||
|
|
Fixed for build 2315 |
|
|
Added support for GridPopupDelay setting in Options section of MediaMonkey.ini. Default value is 275 (ms). |
|
|
Per IM discussion, an addon enabling the delay seems to be a good solution. |
|
|
I don't see the point why anyone would want to install an addon that enables just a delay? IMHO such an addon will just needlessly occupy our addons repository. I would rather expect an addon to override the double-click behaviour to start playback (like in MM4) ? |
|
|
Ludek: because for people, who use double-click to go to sub-view, and do not use pop-up much, is worse experience to open popup with every double-click, and it can also affect smoothness of going to sub-view |
|
|
And, I suppose that the addon could also evolve, e.g. to let user define double click action as Ludek mentioned. |
|
|
Can do :) Perhaps it can be under Behavioral Tweaks, titled something like "Customize Grid View Behavior"? |
|
|
So can this issue be tagged as resolved in 5.0? i.e. the code changes in MM are all complete in 2315 and ready to be tested (the creation of an addon is independent of this) ? |
|
|
I would say so, yes. We can make a new issue to discuss the addon EDIT: I don't think we came to an agreement on how long the delay should be by default. |
|
|
I tested on a current asus core i3 laptop and a 5-year old dell celeron laptop, using mouse, touchpads, and touchscreens. In all cases 175ms seemed to work well, so I'd set that as the default. Note: 150ms was too fast for the touchpad on one of the laptops (and 0ms, though faster results in flashing of the popup prior to opening the new screen. A more permanent/future solution could eliminate the performance/UI compromises e.g. current approach of allow links and popups b) allowing only links c) allowing only popups. |
|
|
Fixed in 2316, default delay changed to 175ms. I did not remove possibility of changing this in ini setting, so advanced users like drakinite still can change this. In case you already changed this setting to different value, the new default will not by applied. |
|
|
Verified 2316. |
