Skip to content

Conversation

@legend-df
Copy link
Owner

Summary

  • introduce a FlowchartManager widget that allows multiple calculator flowcharts to be managed in a single window
  • update the calculator example to use the manager and seed multiple tabs on startup
  • register the new sources in the calculator CMake target so the new widget is built

Testing

  • cmake -S . -B build (fails: missing Qt and unable to download Catch2 in the environment)

https://chatgpt.com/codex/tasks/task_e_68eda61890e4832d99c8d679f7caa1b2

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

Comment on lines +36 to +38
add_executable(headless_calculator
${HEADLESS_CALC_SOURCE_FILES}
${CALC_HEAEDR_FILES}
${CALC_HEADER_FILES}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Avoid moc’ing FlowchartManager in headless target

The shared ${CALC_HEADER_FILES} list now contains FlowchartManager.hpp, which declares FlowchartManager and FlowchartPage with Q_OBJECT. Reusing that list for headless_calculator causes CMake’s automoc to generate meta-object code for those classes, but the target never builds FlowchartManager.cpp, so the moc object will reference undefined symbols such as FlowchartManager::addFlowchart() at link time. This breaks the headless example even though it never uses the flowchart manager. Consider either removing FlowchartManager.hpp from the header list or excluding it from the headless target to keep the build linking.

Useful? React with 👍 / 👎.

legend-df and others added 27 commits October 14, 2025 09:43
…charts

Add execution controls for calculator flowcharts
修改CMakeLists.txt,将默认的USE_QT6选项设为OFF以使用Qt5,并添加C++17标准要求。同时更新Catch2下载配置,增加TLS验证关闭和超时设置。
…for-calculator-buttons

Add manual execution control to calculator flowcharts
…d-issue-gcvabz

Improve flowchart save/load workflow
…ad-issue-5974lf

Improve multi-flowchart save and load handling
…ad-issue-l489wv

Fix Windows build issues for flowchart manager
…ad-issue-eg36j4

Add project save/load workflow to calculator example
…ad-issue-83s46s

Fix calculator build issues on Windows
…ad-issue-oco3km

Rename FlowchartPage save helper for Windows build
…ad-issue-za2tzd

Restore FlowchartPage::saveAs wrapper for compatibility
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants