Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

colors do not show up in the (color label) context menu #569

Closed
burg1ar opened this issue May 18, 2023 · 8 comments
Closed

colors do not show up in the (color label) context menu #569

burg1ar opened this issue May 18, 2023 · 8 comments
Assignees
Labels
status: ready for testing resolved but needs testing type: bug
Milestone

Comments

@burg1ar
Copy link

burg1ar commented May 18, 2023

image

klogg version: klogg 22.06.0.1289
os: ubuntu 22.04

this seems to be a bug according to #270

@variar
Copy link
Owner

variar commented May 27, 2023

Unfortunately, this seems to be a Qt 5 issue in Linux. For Qt6 it works fine, so I've enabled it for Qt6 builds

@variar variar self-assigned this May 27, 2023
@variar variar added this to the 2023.next milestone May 27, 2023
@variar variar added the status: ready for testing resolved but needs testing label Jul 1, 2023
@nowhszh
Copy link

nowhszh commented Jul 4, 2023

QPixmap pixmap( 20, 10 );
auto fillColor = currentLabelConfiguration.color.backColor;
fillColor.setAlphaF( 1.0 );
pixmap.fill( fillColor );
colorLabelAction->setIcon( QIcon( pixmap ) );
#if defined( Q_OS_WIN ) || QT_VERSION >= QT_VERSION_CHECK( 6, 0, 0 )
colorLabelAction->setIconVisibleInMenu( true );
#endif

I do not understand the problem, comment out lines 699 and 701, using Qt5.12 compile, the color display is normal

@nowhszh
Copy link

nowhszh commented Jul 4, 2023

I will verify this again using branch tag v22.06

@nowhszh
Copy link

nowhszh commented Jul 4, 2023

@variar

QPixmap pixmap( 20, 10 );
auto fillColor = currentLabelConfiguration.color.backColor;
fillColor.setAlphaF( 1.0 );
pixmap.fill( fillColor );
colorLabelAction->setIcon( QIcon( pixmap ) );
#ifdef Q_OS_WIN
colorLabelAction->setIconVisibleInMenu( true );
#endif

The code in tag v22.06 only displays color icons in klogg for Windows platform. setIconVisibleInMenu function was introduced in Qt4.4, I actually tested in Mac and Ubuntu are working properly, why is it enabled only in Windows?

@nowhszh
Copy link

nowhszh commented Jul 4, 2023

I will verify this again using branch tag v22.06

After commenting out the code it works fine

@variar
Copy link
Owner

variar commented Jul 5, 2023

There were some rendering issues with Qt5 and KDE setup. If I remember correctly there were black squares instead of colors. I can't reproduce it anymore, something in my system got updated and fixed the issue. I guess it could be enabled back on older versions of Qt.

@nowhszh
Copy link

nowhszh commented Jul 6, 2023

@variar I also have no problems with multiple computers, and I also think I can re-enable

@variar
Copy link
Owner

variar commented Jul 10, 2023

Enabled for all builds in 23.06.0.1538+

@variar variar closed this as completed Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: ready for testing resolved but needs testing type: bug
Projects
None yet
Development

No branches or pull requests

3 participants