qml-colorpicker icon indicating copy to clipboard operation
qml-colorpicker copied to clipboard

Runtime error: TypeError: Cannot read property 'pixelSize' of undefined

Open Kicer86 opened this issue 1 year ago • 6 comments

I built and ran test:

$ ./test 
qml: v:0
qml: handle signal:#000000
qml: handle signal:#00000000
qml: v:NaN
qml: hue value is outside of expected boundaries of [0, 1]
qml: handle signal:#00000000
qml: v:0
qml: handle signal:#00000000
qml: v:NaN
qml: hue value is outside of expected boundaries of [0, 1]
qml: handle signal:#00000000
qml: v:0
qml: handle signal:#00000000
qml: handle signal:#000000
qml: handle signal:#000000
qrc:/content/SizeConfigrator.qml:155: TypeError: Cannot read property 'pixelSize' of undefined
qrc:/Colorpicker.qml:255: ReferenceError: numberBoxFontPixelSize is not defined
qrc:/Colorpicker.qml:262: ReferenceError: numberBoxFontPixelSize is not defined
qrc:/Colorpicker.qml:269: ReferenceError: numberBoxFontPixelSize is not defined

And this is how it looks: obraz

My environment:

$ qmake6 -v
QMake version 3.1
Using Qt version 6.5.2 in /usr/lib

Kicer86 avatar Aug 15 '23 07:08 Kicer86

I tried to test with Qt6.5.2 on Windows10. As the result the test was failure. It seemed that the reason was different from yours, in any case I will try to fix this issue. For my reference, let me know your OS and Compiler.

BTW, the latest of my pull-request and this issue are unrelated, I guessed.

KouOuchi avatar Aug 16 '23 05:08 KouOuchi

I'm using gcc (GCC) 13.2.1 20230801 on Arch Linux

Kicer86 avatar Aug 16 '23 11:08 Kicer86

Sorry for late reply! I tried on my latest PQ branch, the result seemed good.

Please try the PQ branch : KouOuchi:change_layout_bugfix_pq1

Note : Please apply below patch on Linux if you have something trouble with building. I tried on Ubuntu 22.04 / Qt6.5.2.

1 file changed, 8 insertions(+), 3 deletions(-)
test/CMakeLists.txt | 11 ++++++++---

modified   test/CMakeLists.txt
@@ -50,6 +50,9 @@ if(WIN32)
 elseif(ANDROID)
   qt_add_library(${PROJECT_NAME} ${SOURCE_FILES} ${HEADER_FILES} SHARED)
   set_property(TARGET ${PROJECT_NAME} PROPERTY QT6_NO_LINK_QTMAIN ON)
+elseif(UNIX AND NOT APPLE)
+    qt6_add_executable(${PROJECT_NAME} ${SOURCE_FILES} ${HEADER_FILES}
+      ${RCC_SOURCES})
 endif(WIN32)
 
 include_directories(
@@ -76,9 +79,11 @@ if (MSVC)
 #    set (QT_PLUGINS_DIR "${QT_BINARY_DIR}/../plugins")
 #  endif()
   #configure_file("${CMAKE_SOURCE_DIR}/adm/cmake/sample.vcxproj.user.in" "${CMAKE_BINARY_DIR}/${PROJECT_NAME}/${PROJECT_NAME}.vcxproj.user" @ONLY)
-endif()
 
-add_custom_command(TARGET ${PROJECT_NAME} POST_BUILD 
+  add_custom_command(TARGET ${PROJECT_NAME} POST_BUILD
     COMMAND Qt6::windeployqt
     ARGS $<TARGET_FILE:${PROJECT_NAME}>
-)
+  )
+
+endif()
+

KouOuchi avatar Sep 15 '23 10:09 KouOuchi

It works (and your build patch was needed)

Kicer86 avatar Sep 20 '23 19:09 Kicer86

Thank you for your confirmation.

BTW, I will also try to investigate build failure on Win+Qt6.5.2. I'm thinking of re-creating the pull request based on that.

KouOuchi avatar Sep 21 '23 00:09 KouOuchi

Hey @KouOuchi, sorry for not closely following this through recently.

Thank you so much for all your contributions, I've added you as a core contributor to the project (I believe I did, at least).

rshest avatar Sep 24 '23 03:09 rshest