Menu

#943 Use system hotlight color for start page links

Next_Nightly
open
Patch
2020-04-18
2020-04-06
No

Discussion

  • Michel Le Bihan

    Michel Le Bihan - 2020-04-07

    Sorry. I forgot the patch.

     
  • Michel Le Bihan

    Michel Le Bihan - 2020-04-10

    @bluehazzard Is this patch OK?

     
  • Teodor Petrov

    Teodor Petrov - 2020-04-10

    Probably, I'll apply it soon.

     
  • Teodor Petrov

    Teodor Petrov - 2020-04-10
    • assigned_to: Teodor Petrov
    • Milestone: Undefined --> Next_Nightly
     
  • Teodor Petrov

    Teodor Petrov - 2020-04-17

    This doesn't work for wxGTK wx-master and gtk2. The link colour is black for me.
    What is the version of wxGTK and GTK you're using?

     

    Last edit: Teodor Petrov 2020-04-17
  • Teodor Petrov

    Teodor Petrov - 2020-04-17

    It is not surprising given the code in src/gtk/settings.cpp in wx-master looks like this:

            case wxSYS_COLOUR_HOTLIGHT:
                // TODO
                color = *wxBLACK;
                break;
    
     
  • Michel Le Bihan

    Michel Le Bihan - 2020-04-18

    @fuscated Sorry. I tested it only on GTK3 where the color is as expected. Will it be OK to set wxBLUE for GTK2 and wxSYS_COLOUR_HOTLIGHT for GTK3?

     
  • Teodor Petrov

    Teodor Petrov - 2020-04-18

    Someone should check the situation on the other two platforms - MSW and Cocoa.

    Or someone should patch wxWidgets, to fix GTK there...

     

    Last edit: Teodor Petrov 2020-04-18
  • Teodor Petrov

    Teodor Petrov - 2020-04-18
    • labels: --> UI, start_here
     
  • Miguel Gimenez

    Miguel Gimenez - 2020-04-18

    In MSW 10 the change works OK, no difference

     
  • Teodor Petrov

    Teodor Petrov - 2020-04-18

    Ok, it seems it is not too hard to fix https://github.com/wxWidgets/wxWidgets/pull/1805

     
  • Michel Le Bihan

    Michel Le Bihan - 2020-04-18

    Even if it's fixed in wxWidgets, to won't help Code::Blocks much because it will take ages for the change to make it to stable distributions. On top of that, distros refuse to package wx 3.1 because the API is not stable.

     
  • Teodor Petrov

    Teodor Petrov - 2020-04-18

    This is true. On wxCocoa the color returned by this is white when using the dark theme. So it seems broken there, too. :(

        case wxSYS_COLOUR_HOTLIGHT:
            // OSX doesn't change color on mouse hover
            sysColor = [NSColor controlTextColor];
            break;
    
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.