Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

Post a reply

Smilies
:D :) :( :o :-? 8) :lol: :x :P :oops: :cry: :evil: :roll: :wink:

BBCode is ON
[img] is ON
[url] is ON
Smilies are ON

Topic review
   

Expand view Topic review: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

Re: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

by drakinite » Tue Jun 30, 2020 9:42 pm

Ah, okay. That's not surprising, seeing as they aren't used much.
It is still possible to update the tile icon programmatically, however! (and it would probably be more stable than using live tiles)

First, the start menu shortcut to MM5 would have to be inside the AppData directory instead of the ProgramData directory, because modifying shortcut files that exist in ProgramData requires admin permissions. (For example: keep MM5 shortcuts in "C:\Users\USER\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\MediaMonkey 5" instead of where it currently resides, which is "C:\ProgramData\Microsoft\Windows\Start Menu\Programs\MediaMonkey 5")

After updating MediaMonkey.VisualElementsManifest.xml to the new logo file location and the new background color, all you have to do is modify the lastwritetime property of MediaMonkey 5.lnk to the current time, and Windows will automatically refresh the start menu tile.

I did it manually in PowerShell with this command:

Code: Select all

(ls "$env:appdata\microsoft\windows\start menu\programs\MediaMonkey.lnk").lastwritetime = get-date
....I'm talking to developers, so you guys probably already know all this, don't you? Haha. :sweat_smile:

Re: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

by Lowlander » Tue Jun 30, 2020 8:57 pm

It's my understanding that Live Tiles will be killed by Microsoft.

Re: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

by drakinite » Tue Jun 30, 2020 8:52 pm

Oh gosh, that would be so cool. I don't know how responsive live tiles are in Windows; but if they don't have too much performance overhead and they take less than a second to change, we could even include currently playing song information on live tiles.
But even if that's not an option, updating the tile skin via the Windows live tile function would definitely be a possibility. The second and third links you sent detail stuff that's a bit out of date, for Windows 8; but it might still work on Win10. What runtime/API is MM5 using? (I can see that it's not using Node/Electron.) I'm not quite sure how easily we can get access to those live tiles if it's not a UWP or "Windows Runtime" app.

Re: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

by Peke » Tue Jun 30, 2020 7:44 pm

Re: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

by drakinite » Tue Jun 30, 2020 7:15 pm

Aha, I found out what it is! It looks like the large icon was entirely intentional.

Inside MediaMonkey.VisualElementsManifest.xml:

Code: Select all

<Application xmlns:xsi='http://www.w3.org/2001/XMLSchema-instance'>
  <VisualElements
      ShowNameOnSquare150x150Logo='on'
      Square150x150Logo='Assets\BigLogo.png'
      Square70x70Logo='Assets\StoreLogo.png'
      ForegroundText='light'
      BackgroundColor='#0078d7' />
</Application> 
Lo and behold, BigLogo.png and StoreLogo.png are the full-size icons that were perplexing all of us.

EDIT: Oh, whoops, now I see what you meant in the previous post. I didn't realize you were already aware of the intentional 150x150 png and you were asking for ideas as to why it was showing up that way. Facepalm aside, my suggestion is below:

I believe that Microsoft does a pretty good job at their Square150x150Logo and Square70x70Logo sizing/positioning; Maybe MM's logo could use the same sizing, to help promote a consistent design across the platform. Here, let me put them in this post so you can see:

VS Code:
Image Image
Office: (you can also look in C:\Program Files\Microsoft Office\root\Office16\LogoImages)
Image Image Image Image

...Also.. also.. ALSO–
I have a CRAZY idea.
What if different skins can have access to the logo files and/or background color? So if you use the Material Dark, the logo changes to be black and orange, but if you use Black Monkey, it goes to brown and light yellow, etc.
Here's an example of what I'm imagining:
Image Image
And yes- I will be setting the logo/background to that of the first image for my own install. I like it too much. ;P

Re: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

by Peke » Tue Jun 30, 2020 6:19 pm

Hi,
Bugs [#16763][#16764] should be fixed in next MM5 version (build 2259+)

Re: Logging out causes program memory error; plus 7 other bugs/suggestions! [#16763][#16764][#16234]

by Peke » Tue Jun 30, 2020 5:27 pm

Hi,
Re Icon: MM5 Icon is made according to MS dimensions 150x150 along with latest windows specs and many other icons are still up to 96x96 :) So as Windows itself controls the size of Icon in Tile it shows best one that fit :) Do you have any suggestions?

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by drakinite » Tue Jun 30, 2020 3:40 pm

MiPi wrote: Tue Jun 30, 2020 6:31 am Problem during logout/shuthown already tracked as: https://www.ventismedia.com/mantis/view.php?id=16234
It is hard to find one. Strange also is, that it displays 64b memory addresses, but MM5 is 32bit application...
Wow, weird. Well, at least it's being tracked! :)

MiPi wrote: Tue Jun 30, 2020 6:49 am Double-click column filter added as: https://www.ventismedia.com/mantis/view.php?id=16764
MiPi wrote: Tue Jun 30, 2020 6:13 am Hi. I have found how to reproduce problem with tooltips, tracked as: https://www.ventismedia.com/mantis/view.php?id=16763
Sweet!


MiPi wrote: Tue Jun 30, 2020 6:31 am
I don't see problem with tooltip frame rate or CPU load at all, for me it is fluent and it takes around 10% CPU during playback and mouse hovering. (i5-6500 with onboard GPU)

What do you mean by "scrubber"/"scrub bar"?
By "scrub bar" I mean this thing- The bar that displays the percentage of the song and that lets you click to change what time of the song that is playing. I honestly don't know what else to call it. I recorded a demonstration of what the framerate is like for the song-progress tooltip. The frame rate when the tooltips fade in is very low (under 10 fps). But the song progress tooltip becomes smooth after the fade-in animation has completed. https://puu.sh/G2fwi/456f58ee20.mp4
Image


MiPi wrote: Tue Jun 30, 2020 6:31 am Icons in Windows startmenu have size based on your Windows settings, you can rightclick icon and change its size there.
This is what it looks like - The icon takes up the entire tile, whether the tile is medium or small; but no other program is like that. Strangely, Berni said that the icon in their start menu is the same size as all the others. I don't know what makes it different for me.
Image


Also - Since you're here, there was an additional thread I started a few days ago with another display-related bug. I don't think anyone added it to the developer issues list yet.
https://www.mediamonkey.com/forum/viewt ... 30&t=96860

Thanks for all the help and for listening!

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by MiPi » Tue Jun 30, 2020 6:49 am

Double-click column filter added as: https://www.ventismedia.com/mantis/view.php?id=16764

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by MiPi » Tue Jun 30, 2020 6:40 am

Re frame rate of tooltip - just found out, it will be the same issue as not displaying it, so should be fixed with this then, I need also 125% zoom in Windows to reproduce its problematic display.

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by MiPi » Tue Jun 30, 2020 6:31 am

Problem during logout/shuthown already tracked as: https://www.ventismedia.com/mantis/view.php?id=16234
It is hard to find one. Strange also is, that it displays 64b memory addresses, but MM5 is 32bit application...

I don't see problem with tooltip frame rate or CPU load at all, for me it is fluent and it takes around 10% CPU during playback and mouse hovering. (i5-6500 with onboard GPU)

What do you mean by "scrubber"/"scrub bar"?

Icons in Windows startmenu have size based on your Windows settings, you can rightclick icon and change its size there.

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by MiPi » Tue Jun 30, 2020 6:13 am

Hi. I have found how to reproduce problem with tooltips, tracked as: https://www.ventismedia.com/mantis/view.php?id=16763

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by drakinite » Mon Jun 29, 2020 2:17 pm

That explains your lower CPU usage; I have an i7-6700hq which is less than half as powerful as yours.

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by berni » Mon Jun 29, 2020 12:50 pm

My system:
Image

Something is different on your PC.
For me, mouseover and the icon works fine.
Berni

Re: Logging out causes program memory error; plus 7 other bugs/suggestions!

by drakinite » Mon Jun 29, 2020 10:27 am

What CPU do you have?
Also, I have tooltips enabled, yet they don't work right. I tried changing to Black Monkey but got the same experience. Interestingly, when I recorded this, the first time I moused over the shuffle button it DID show a shuffle tooltip. That was the first time I ever saw the tooltip, but it didn't show up again. https://puu.sh/G1NgG/cd3d6d6c2b.mp4
As for the icon size, well, that's really weird. Honestly I'm confused as to how the icon could possibly be that large on my computer, because I thought it was impossible.

Top