Status icons #142

Merged

zach merged 2 commits from refactor/status-icons into main 2026-08-11 15:03:04 +02:00

Conversation 0 Commits 2 Files Changed 20

+1321 -324

zach commented 2026-08-11 14:08:36 +02:00

Owner

Copy Link

Copy Source

What & why

Previously I had nested the status icons (audio, mic, battery) into its own separate container which had its own helper method to find popouts and then open them. This removes the cluttered separation and makes it cleaner, keeping it all directly available from Bar.qml.

It also fixes an issue where the animation curve values and related were written to the config file.

Related issue

No response

Type

Refactor / style

Tested

• Build succeeds
• Existing tests pass
• Manually verified

Breaking changes?

No response

Checklist

• CI passes (format, lint, typecheck)
• Clean commits
• No new warnings
• Docs updated if needed

### What & why Previously I had nested the status icons (audio, mic, battery) into its own separate container which had its own helper method to find popouts and then open them. This removes the cluttered separation and makes it cleaner, keeping it all directly available from `Bar.qml`. It also fixes an issue where the animation curve values and related were written to the config file. ### Related issue _No response_ ### Type Refactor / style ### Tested - [x] Build succeeds - [x] Existing tests pass - [x] Manually verified ### Breaking changes? _No response_ ### Checklist - [x] CI passes (format, lint, typecheck) - [x] Clean commits - [x] No new warnings - [x] Docs updated if needed

zach added 1 commit 2026-08-11 14:08:37 +02:00

refactor: use entries directly in bar object instead of nested components + various fixes and additions to settings d81808dd8f

zach added 1 commit 2026-08-11 15:00:52 +02:00

simpler model for status entries 2e4517b0a6

zach merged commit 7482741e46 into main 2026-08-11 15:03:04 +02:00

zach deleted branch refactor/status-icons 2026-08-11 15:03:05 +02:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

bug

Something isn't working

documentation

Improvements or additions to documentation

duplicate

This issue or pull request already exists

enhancement

New feature or request

good first issue

Good for newcomers

help wanted

Extra attention is needed

invalid

This doesn't seem right

question

Further information is requested

wontfix

This will not be worked on

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

AramJonghu Inorishio zach

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: zach/z-bar-qt#142