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

[raster] New single color renderer #57074

Merged
merged 7 commits into from
Apr 8, 2024
Merged

Conversation

nirvn
Copy link
Contributor

@nirvn nirvn commented Apr 7, 2024

Description

This PR implements a single color renderer for raster layers. This allows for a super simple and friendly way to render a raster using a single color:

Screencast.from.2024-04-08.12-47-22.mp4

@nirvn nirvn added Feature Rasters Related to general raster layer handling (not specific data formats) labels Apr 7, 2024
@github-actions github-actions bot added this to the 3.38.0 milestone Apr 7, 2024
src/core/raster/qgssinglecolorrenderer.h Outdated Show resolved Hide resolved
src/core/raster/qgssinglecolorrenderer.h Outdated Show resolved Hide resolved
src/core/raster/qgssinglecolorrenderer.h Outdated Show resolved Hide resolved
src/core/raster/qgssinglecolorrenderer.h Outdated Show resolved Hide resolved
src/core/raster/qgssinglecolorrenderer.h Outdated Show resolved Hide resolved
src/core/raster/qgssinglecolorrenderer.cpp Outdated Show resolved Hide resolved
src/core/raster/qgssinglecolorrenderer.cpp Outdated Show resolved Hide resolved
src/core/raster/qgssinglecolorrenderer.cpp Outdated Show resolved Hide resolved
src/gui/raster/qgssinglecolorrendererwidget.h Outdated Show resolved Hide resolved
src/gui/raster/qgssinglecolorrendererwidget.h Outdated Show resolved Hide resolved
@nyalldawson
Copy link
Collaborator

Nice work!

I think we'll also need to expose methods to set the desired input band here, and add a band selection combo to the renderer's widget.

@nirvn
Copy link
Contributor Author

nirvn commented Apr 8, 2024

@nyalldawson , yep, it occurred to me we still need a band selection. It'll come today.

@nirvn nirvn merged commit 35aadbc into qgis:master Apr 8, 2024
30 checks passed
@DelazJ DelazJ added the Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo. label Apr 11, 2024
@qgis-bot
Copy link
Collaborator

@nirvn
This pull request has been tagged as requiring documentation.

A documentation ticket will be opened at https://github.com/qgis/QGIS-Documentation when this PR is merged.

Please update the description (not the comments) with helpful description and screenshot to help the work from documentors.
Also, any commit having [needs-doc] or [Needs Documentation] in will see its message pushed to the issue, so please be as verbose as you can.

Thank you!

@qgis-bot
Copy link
Collaborator

@nirvn
A documentation ticket has been opened at qgis/QGIS-Documentation#9000
It is your responsibility to visit this ticket and add as much detail as possible for the documentation team to correctly document this change.
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Needs Documentation When merging a labeled PR, an issue will be created in the Doc repo. Rasters Related to general raster layer handling (not specific data formats)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants