Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

refactor: delete the dead StatusToForegroundValueConverter

Open Beginner friendly
  • Is something wrong?
#342 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
1-3 hours
Newbie friendliness
91/100
Issue type
Refactor
Clarity
Clearly specified
Activity status
Active
Tech stack
csharp
Domain
desktop

Research direction

Delete src/LageBuch.App.Shared/Converters/StatusToForegroundValueConverter.cs and check whether anything remains in Converters/; remove the folder if it is empty. Then run make build and make test to catch any references not found by the reported search. Done when both commands pass and the unused converter is gone.

Written by the indexing model from the issue text.

Description

enhancement good first issue

src/LageBuch.App.Shared/Converters/StatusToForegroundValueConverter.cs is dead
code, and it is also broken code.

A repo-wide search for StatusToForeground across src/ and tests/, in both
.cs and .axaml files, matches only the declaration itself. Nothing binds to
it. On top of that its Convert ignores the value argument entirely and
always returns a black brush, which on this dark theme would be invisible if it
ever were used.

What to do

Delete the file. Check whether Converters/ still holds anything afterwards and
remove the folder if it does not.

Then build and run the tests to confirm nothing referenced it through a string
in XAML that a grep would miss.

Verify
make build
make test

Difficulty: easy, roughly ten minutes. A clean first pull request.

New here?

Comment to claim this issue and it is yours. If no pull request appears within
seven days it goes back in the pool, no hard feelings.

Before you open the PR, please skim CONTRIBUTING.md:
commits need DCO sign-off (git commit -s) and a Conventional Commit subject.
Changes under src/ additionally need an entry under ## [Unreleased] in
CHANGELOG.md — CI fails the PR otherwise.

Questions are welcome right here or in
Discussions.

Dominant language
C#
Stars
1
Forks
3
Avg merge
4h 34m
Merged PRs (30d)
182

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from CodeForFire/lagebuch

All issues in CodeForFire/lagebuch

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.