1349: pulse icons messed up in properties window [#7422]

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: 1349: pulse icons messed up in properties window [#7422]

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Sun May 01, 2011 9:44 am

Fixed in 1368.

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Fri Apr 22, 2011 6:05 am

PetrCBR wrote:I can't see any problem with Pulse skin ... can you give me any screenshot with a description what's wrong ?
HI Petr,

Here it is. ( Icons on the right side shows correct. )
Image

Re: 1349: pulse icons messed up in properties window [#7422]

by PetrCBR » Tue Apr 19, 2011 2:16 pm

I can't see any problem with Pulse skin ... can you give me any screenshot with a description what's wrong ?

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Tue Apr 19, 2011 8:22 am

Problem still present in 1365

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Sat Apr 16, 2011 7:22 am

That's because I used a screenshot from gege of a previous build.

But the problem is still in 1363. The icons looks aliased because the alpha channel is broken.
It's the same issue that happened with the other icons in this thread. But I didn't notice that it was broken in that particular dialog till now.

Re: 1349: pulse icons messed up in properties window [#7422]

by PetrCBR » Tue Apr 12, 2011 1:56 pm

Are you sure your screenshot is from 1363 ? I've tried PTBR with latest build and there's much more space for checkboxes than in your shots.
Also what icons problem ? I don't see any difference between them.

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Tue Apr 12, 2011 1:26 am

1363 pulse skin: the icon in front of "Scan continuously"
and the one in front of "Scan at startup" are not rendered correctly.
Image

Could someone reopen the ticket with this new information?

Thank you.

Re: 1349: pulse icons messed up in properties window [#7422]

by nohitter151 » Wed Mar 02, 2011 8:00 pm

Dreadlau wrote:...because of the anti-aliasing rendering issue I reported in this thread. And it looks ok now?
Yes.

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Wed Mar 02, 2011 7:52 pm

So to be clear. Does the design of the "notepad" icon look bad?
Or was it because of the anti-aliasing rendering issue I reported in this thread. And it looks ok now?

As for the different icon. I think it's worth to be reported. A collection and a statistic/text file is two different things.

Re: 1349: pulse icons messed up in properties window [#7422]

by nohitter151 » Wed Mar 02, 2011 7:26 pm

Dreadlau wrote:I'm not following you. :-?
So my statistic icon looks outdated and aliased?
It has the same gradient/style/borders/anti-aliasing/shadow than any other icons in the pulse set.

Are you sure that's what he meant?
He said mine was outdated and yours looked bad (not antialiased) in the older build. You even showed that in the first post of this thread. http://www.mediamonkey.com/forum/viewto ... 76#p288331

And yes, I'm sure. I exchanged emails with him about it.

Anyways if you think we need a new icon, we should open a new bug for it. But I don't really see that it's necessary. Even your icon works for both locations (the notepad one).

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Wed Mar 02, 2011 7:20 pm

I'm not following you. :-?
So my statistic icon looks outdated and aliased?
It has the same gradient/style/borders/anti-aliasing/shadow than any other icons in the pulse set.

Are you sure that's what he meant?

Re: 1349: pulse icons messed up in properties window [#7422]

by nohitter151 » Wed Mar 02, 2011 7:10 pm

He meant update it because it looked bad (eg outdated/antialiasing issues).

Anyway I think my icon fits well enough for both.

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Wed Mar 02, 2011 6:58 pm

It is mentioned by rusty in the ticket:
rusty wrote:a) Stephen, Dreadm:
-Can you update the Edit Properties icon ?
And I agree it needs update and I provided a replacement icon with a download link.
nohitter151 wrote:I don't really have a problem with using the same icon for both.
Well, I do have a problem with that.

There is no similarity between "statistics" and "collections" icons.

Right now "edit collection" use an icon that doesn't represent the function.
Which is a basic requirement for any program interface!

Re: 1349: pulse icons messed up in properties window [#7422]

by nohitter151 » Wed Mar 02, 2011 5:55 pm

There was no bug assigned to that issue, I don't really have a problem with using the same icon for both.

Re: 1349: pulse icons messed up in properties window [#7422]

by Dreadlau » Wed Mar 02, 2011 4:19 pm

Still not fixed in build 1351.

"New collection" item is still using an unadapted icon!

We need a new icon object for "new collection"

See my previous post :roll:

Top