Repository navigation
feat: phase 2 stratigraphic column changes - #133
Open
lachlangrose wants to merge 7 commits into
Open
lachlangrose wants to merge 7 commits into
lachlangrose wants to merge 7 commits into
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
QTimer.singleShot(0, func) runs func on the calling thread, not the Qt main thread. A log message with push=True from a background task (for example the thickness calculator) changed the QGIS message bar from the worker thread, and QGIS stopped responding when the task ended. Send the message bar change to the main thread with a queued signal.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…umn (#129) * feat: highlight the selected stratigraphic unit on the map Select a unit row in the stratigraphic column to select the features of that unit in the units layer, so that QGIS highlights them on the map. The match uses the unit-name field set under the column. Select an unconformity row, or clear the selection, to remove the highlight. Focus on a field of a unit row also selects that row, because clicks on the row's fields do not reach the list. * feat: apply stratigraphic thickness to the map layer Add a button to the stratigraphic column that writes a 'strat_thickness' field onto the selected units layer. The field holds the thickness of the matching unit in the column. Features that do not match a unit stay empty. The layer then gets a graduated renderer with one class for each thickness value, coloured with the selected colour ramp. Move the colour ramp lookup into a helper, so that the age and thickness styling use the same lookup. * style: apply black formatting
Put the geology layer first with a summary of the unit names that have no match. Use text buttons for adding units, a Build column menu and an overflow menu. Add Youngest and Oldest labels and a text for an empty column. Replace the three apply buttons with one Style map layer group. Use QgsColorButton and theme icons in the rows, and remove the outer scroll area.
lachlangrose
added this pull request to stack #139
October 7, 2026 14:06
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Phase 2 of the usability plan. It is based on phase 1 (#132), so the base branch is
feat/shared-layer-roles. Change the base tomainafter #132 is merged.QgsColorButtonand a theme delete icon, and have no repeated "Thickness:" label. Unconformity rows have a different background.main, which has the map highlight and thickness styling (feat(ui): highlight the stratigraphic unit on map when clicked in column #129).docs/usage/interface.mdand the plan are updated.Test plan
🤖 Generated with Claude Code