Bug #1463: Seeing both growl and synergy notification window when changing ratings

Comments

1. Greg Hurrell 2010-01-21T02:20:43Z

   Growl is controlled completely independently in its own preference pane (in the System Preferences application).

   Synergy's Floater is controlled in the Synergy Preferences, in the "General" tab (the first one you see when you open the preferences).

   There you'll find a checkbox to turn it on and off, as well as checkboxes to control what information gets displayed.

   If you think there's a glitch in the Floater (ie it's missing information) and you're sure the checkboxes are correctly set, I'd recommend that you try stopping Synergy, moving your preferences file out of the way (from ~/Library/Preferences/org.wincent.Synergy.plist) and see if the problem goes away. (Corrupt prefs files are the most common cause of odd behaviour.)

2. zbeckman 2010-01-21T13:17:01Z

   I tried deleting the preference pane, but no good.

   Synergy Preference has the "Display floater when iTunes track changes" item UNCHECKED, so it should not show any floater, ever.

   But, it ALWAYS shows the rating change floater -- but nothing else. Otherwise it behaves as expected.

3. zbeckman 2010-01-21T13:18:06Z

   Oops, I mean "I tried deleting the preferences file..."

4. Greg Hurrell 2010-01-21T14:34:48Z

   Ok, I think I misinterpreted you.

   I originally thought you were talking about the Floater, but I now think you are talking about the notification bezels that appear in the middle of the screen and which look like the bezels you see when you use Mac OS X's built-in brightness and volume function keys.

   If that's what you want to turn off, the checkbox you need to hit is the one which says "Extra visual feedback for next, previous etc". That'll turn off all such bezels.

5. zbeckman 2010-01-21T17:47:27Z

   That checkbox is ALSO unchecked. I had the same thought...

6. Greg Hurrell 2010-01-22T02:13:53Z

   Just to be clear what's happening here, can you take a screenshot and mail it to me so that I can see what you're seeing?

   In case you don't know how to take a screenshot, you can do so by pressing and holding Command + Shift + 3 momentarily. The screenshot file will be saved to your desktop.

7. anonymous 2010-02-03T16:11:59Z

   Just sent you a screenshot. Here's a link: http://skitch.com/zbeckman/nuuba/screen-shot-2010-02-03-at-4.06.23-pm

8. Greg Hurrell 2010-02-04T02:23:59Z

   Thanks a lot for the screenshot. Certainly strikes me as very odd that you're seeing that even though you have the "Extra visual feedback for next, previous etc" checkbox unchecked.

   I'm going to see if I can reproduce here locally what you're seeing on your machine.

9. Greg Hurrell 2010-02-04T03:36:07Z

   Good, I can reproduce it. Will try to get a fix ready for next point release (4.3.1).

10. Greg Hurrell 2010-02-06T18:47:34Z

    Funnily enough, digging through the source code I can see from comments like this one...

    // we always show the rating feedback floater, regardless of prefs

    ... that this behavior was actually intentional.

    I can't really remember what my reasons were for taking this stand at the time, but I think there's probably a case for making this bezel fall in line with the other ones.

    Other bezels which are unconditionally shown are ones like iTunes volume up/down bezel, but that one is not so much of a problem seeing as it doesn't seem like a duplicate notification for Growl users.

    I'm just going to leave this one sit for a day or two while I think about what the best course of action to take here is: either sweeping changes to all such bezels, no changes, separate options to control each type of notification, or some other "in between" approach.

11. Greg Hurrell 2010-02-07T08:44:06Z

    I'm thinking that the best thing to do will be to add another checkbox. So we would have the existing "Extra visual feedback for play, pause, next and previous track actions" checkbox, plus a new one "Extra visual feedback for other hot keys".

12. Greg Hurrell 2010-02-09T16:35:28Z

    Ok, I've added this checkbox. It will be in the next release (4.4b).

13. Greg Hurrell 2010-02-09T16:35:38Z

    Status changed:

    • From: new
    • To: closed

14. zbeckman 2010-02-16T19:45:11Z

    Thank you! It's the niggling little inconsistencies that just bug the heck out of me, you know? ;-) Appreciate you taking the time to fix... And you have good commenting style. Document what you did. Beats the "// fix this hack before a client sees it" comment that seems to show up at the worst possible times...

15. Greg Hurrell 2010-02-22T03:04:18Z

    4.4b is now out.

Add a comment

Comments are now closed for this issue.