Skip to content

fix: add the missing "Segment Fill..." entry to the Settings menu - #19

Open
doha-230 wants to merge 1 commit into
akhuettel:masterfrom
doha-230:pr/segment-fill-settings-menu
Open

doha-230 wants to merge 1 commit into
akhuettel:masterfrom
doha-230:pr/segment-fill-settings-menu

Conversation

@doha-230

Copy link
Copy Markdown

The Segment Fill settings dialog is fully wired up - the QAction is created in CreateActions::createSettings, it is connected to
MainWindow::slotSettingsSegments, and the dialog instance is created in CreateSettingsDialogs - but m_actionSettingsSegments is never added to any menu or toolbar, so the dialog cannot be opened at all and the segment fill settings (minimum length, point separation, fill corners, line size, line colour) cannot be changed from the user interface.

The action is added to the Settings menu, next to the Point Match entry, as suggested in issue #9.

The dialog text is unchanged and already translated, so the translation files need no update.

Fixes #9

The Segment Fill settings dialog is fully wired up - the QAction is created in
CreateActions::createSettings, it is connected to
MainWindow::slotSettingsSegments, and the dialog instance is created in
CreateSettingsDialogs - but m_actionSettingsSegments is never added to any menu
or toolbar, so the dialog cannot be opened at all and the segment fill settings
(minimum length, point separation, fill corners, line size, line colour) cannot
be changed from the user interface.

The action is added to the Settings menu, next to the Point Match entry, as
suggested in issue akhuettel#9.

The dialog text is unchanged and already translated, so the translation files
need no update.

Fixes akhuettel#9

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Engauge Digitizer Segements Fill

1 participant