Some suggestions for your nice 3stated widget #2
Conversation
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
|
🧙 Sourcery hat die Überprüfung deines Pull Requests abgeschlossen! Tipps und BefehleInteraktion mit Sourcery
Dein Erlebnis anpassenRufe dein Dashboard auf, um:
Hilfe erhalten
Original review guide in English🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey – ich habe zwei Probleme gefunden und ein paar Hinweise auf höherer Ebene hinterlassen:
- In
configure()ist im Zweig für die Standard-Schwellenwerte die Reihenfolge vonthresholdUp/thresholdDownim Vergleich sowohl zum vorherigen Code als auch zum dynamischen Range-Zweig vertauscht; halte die Feldreihenfolge nach Möglichkeit konsistent, um Verwirrung oder unerwartete Probleme bei der Konfigurationspersistenz zu vermeiden. - Das neue Footer-Rechteck in
w_paint.footerübergibtwidget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET)als Höhen-Argument, was ein absoluter unterer Koordinatenwert und nicht die Footer-Höhe ist; das zeichnet wahrscheinlich mehr als beabsichtigt und sollte stattdessen vermutlichfooterHeightverwenden. - In der neuen Funktion
menu()definierst duCATEGORY_LUAlokal, verlässt dich aber beiCATEGORY_TIMERundCATEGORY_TELEMETRY_SENSORauf globale Variablen; definiere oder importiere alle drei möglichst einheitlich, um versteckte Abhängigkeiten von externen Globals zu vermeiden.
Prompt für KI-Agenten
Bitte bearbeite die Kommentare aus diesem Code-Review:
## Gesamtkommentare
- In `configure()` ist im Zweig für die Standard-Schwellenwerte die Reihenfolge von `thresholdUp`/`thresholdDown` im Vergleich sowohl zum vorherigen Code als auch zum dynamischen Range-Zweig vertauscht; halte die Feldreihenfolge nach Möglichkeit konsistent, um Verwirrung oder unerwartete Probleme bei der Konfigurationspersistenz zu vermeiden.
- Das neue Footer-Rechteck in `w_paint.footer` übergibt `widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET)` als Höhen-Argument, was ein absoluter unterer Koordinatenwert und nicht die Footer-Höhe ist; das zeichnet wahrscheinlich mehr als beabsichtigt und sollte stattdessen vermutlich `footerHeight` verwenden.
- In der neuen Funktion `menu()` definierst du `CATEGORY_LUA` lokal, verlässt dich aber bei `CATEGORY_TIMER` und `CATEGORY_TELEMETRY_SENSOR` auf globale Variablen; definiere oder importiere alle drei möglichst einheitlich, um versteckte Abhängigkeiten von externen Globals zu vermeiden.
## Einzelne Kommentare
### Kommentar 1
<location> `main.lua:362-372` </location>
<code_context>
elseif horizontalAlign == TEXT_RIGHT then
- lcd.drawText(widget.width - wPaint.FREE_RIGHT, textPosY, text, TEXT_RIGHT)
+ lcd.drawText(widget.width - wPaint.FREE_RIGHT - wPaint.TRANSPARENCY_X_OFFSET, textPosY, text, TEXT_RIGHT)
else
lcd.drawText((widget.width / 2), textPosY, text, TEXT_CENTERED)
end
</code_context>
<issue_to_address>
**suggestion (bug_risk):** Die Reihenfolge der Felder thresholdUp/thresholdDown ist zwischen den beiden Zweigen vertauscht.
Wenn sich irgendwelche UI- oder Konfigurations-Tools auf eine stabile Feldreihenfolge verlassen, könnte diese Abweichung zu inkonsistenten Layouts zwischen den beiden Fällen führen. Um subtile UI-Probleme zu vermeiden, gleiche die Einfügereihenfolge in beiden Zweigen an.
```suggestion
-- thresholds
if wHelper.existSource(widget.source) and widget.source.maximum and widget.source.minimum then
-- Not perfect since to have this code working you will need
-- to configure the widget to set the source first
-- and then to configure again to set the remaining parameters
wConfig.addNumberField("thresholdUp", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdDown", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
else
wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
end
```
</issue_to_address>
### Kommentar 2
<location> `lib/w_paint.lua:147` </location>
<code_context>
footerHeight = wPaint.FREE_ABOVE + footerHeight
lcd.color(footerBGColor)
- lcd.drawFilledRectangle(0, widget.height - footerHeight, widget.width, widget.height)
+ lcd.drawFilledRectangle(wPaint.TRANSPARENCY_X_OFFSET, widget.height - footerHeight - (2 * wPaint.TRANSPARENCY_Y_OFFSET), widget.width - (2 * wPaint.TRANSPARENCY_X_OFFSET), widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET))
end
--- draw footer text
</code_context>
<issue_to_address>
**issue (bug_risk):** Das Footer-Rechteck scheint untere Koordinatenwerte zu verwenden, wo Breite/Höhe erwartet werden.
Andere Aufrufe von `lcd.drawFilledRectangle` verwenden die 3. und 4. Parameter als Breite/Höhe, aber hier ist das 4. Argument (`widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET)`) effektiv eine untere Y-Koordinate. In Kombination mit der Reduzierung des Y-Ursprungs um `2 * wPaint.TRANSPARENCY_Y_OFFSET` verschiebt das den Footer vermutlich nach oben und verkleinert ihn fehlerhaft. Bitte ändere diesen Aufruf so, dass der letzte Parameter die Footer-Höhe ist (ähnlich wie bei den anderen Verwendungen) und nicht eine untere Koordinate, z. B.:
```lua
lcd.drawFilledRectangle(
wPaint.TRANSPARENCY_X_OFFSET,
widget.height - footerHeight - wPaint.TRANSPARENCY_Y_OFFSET,
widget.width - (2 * wPaint.TRANSPARENCY_X_OFFSET),
footerHeight
)
```
</issue_to_address>Sourcery ist für Open Source kostenlos – wenn dir unsere Reviews gefallen, teile sie gerne ✨
Original comment in English
Hey - I've found 2 issues, and left some high level feedback:
- In
configure(), the default thresholds branch reverses the order ofthresholdUp/thresholdDowncompared to both the previous code and the dynamic-Range branch; consider keeping a consistent field order to avoid confusion or unexpected config persistence issues. - The new footer rectangle in
w_paint.footerpasseswidget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET)as the height argument, which is an absolute bottom coordinate rather than the footer height; this likely overdraws more than intended and should probably usefooterHeightinstead. - In the new
menu()function you defineCATEGORY_LUAlocally but rely onCATEGORY_TIMERandCATEGORY_TELEMETRY_SENSORas globals; consider defining or importing all three consistently to avoid hidden dependencies on external globals.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In `configure()`, the default thresholds branch reverses the order of `thresholdUp`/`thresholdDown` compared to both the previous code and the dynamic-Range branch; consider keeping a consistent field order to avoid confusion or unexpected config persistence issues.
- The new footer rectangle in `w_paint.footer` passes `widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET)` as the height argument, which is an absolute bottom coordinate rather than the footer height; this likely overdraws more than intended and should probably use `footerHeight` instead.
- In the new `menu()` function you define `CATEGORY_LUA` locally but rely on `CATEGORY_TIMER` and `CATEGORY_TELEMETRY_SENSOR` as globals; consider defining or importing all three consistently to avoid hidden dependencies on external globals.
## Individual Comments
### Comment 1
<location> `main.lua:362-372` </location>
<code_context>
elseif horizontalAlign == TEXT_RIGHT then
- lcd.drawText(widget.width - wPaint.FREE_RIGHT, textPosY, text, TEXT_RIGHT)
+ lcd.drawText(widget.width - wPaint.FREE_RIGHT - wPaint.TRANSPARENCY_X_OFFSET, textPosY, text, TEXT_RIGHT)
else
lcd.drawText((widget.width / 2), textPosY, text, TEXT_CENTERED)
end
</code_context>
<issue_to_address>
**suggestion (bug_risk):** The order of thresholdUp/thresholdDown fields is swapped between the two branches.
If any UI or config tooling relies on a stable field order, this discrepancy could cause inconsistent layouts between the two cases. To avoid subtle UI issues, align the insertion order in both branches.
```suggestion
-- thresholds
if wHelper.existSource(widget.source) and widget.source.maximum and widget.source.minimum then
-- Not perfect since to have this code working you will need
-- to configure the widget to set the source first
-- and then to configure again to set the remaining parameters
wConfig.addNumberField("thresholdUp", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdDown", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
else
wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
end
```
</issue_to_address>
### Comment 2
<location> `lib/w_paint.lua:147` </location>
<code_context>
footerHeight = wPaint.FREE_ABOVE + footerHeight
lcd.color(footerBGColor)
- lcd.drawFilledRectangle(0, widget.height - footerHeight, widget.width, widget.height)
+ lcd.drawFilledRectangle(wPaint.TRANSPARENCY_X_OFFSET, widget.height - footerHeight - (2 * wPaint.TRANSPARENCY_Y_OFFSET), widget.width - (2 * wPaint.TRANSPARENCY_X_OFFSET), widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET))
end
--- draw footer text
</code_context>
<issue_to_address>
**issue (bug_risk):** Footer rectangle seems to be using bottom coordinate values where width/height are expected.
Other `lcd.drawFilledRectangle` calls use the 3rd/4th parameters as width/height, but here the 4th argument (`widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET)`) is effectively a bottom Y coordinate. Combined with reducing the Y origin by `2 * wPaint.TRANSPARENCY_Y_OFFSET`, this likely shifts the footer up and shrinks it incorrectly. Please change this call so the last parameter is the footer height (similar to other usages) rather than a bottom coordinate, e.g.:
```lua
lcd.drawFilledRectangle(
wPaint.TRANSPARENCY_X_OFFSET,
widget.height - footerHeight - wPaint.TRANSPARENCY_Y_OFFSET,
widget.width - (2 * wPaint.TRANSPARENCY_X_OFFSET),
footerHeight
)
```
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
There was a problem hiding this comment.
Hey – ich habe 2 Probleme gefunden und noch ein paar übergeordnete Hinweise hinterlassen:
- Die dynamische Threshold‑Bereichsprüfung in
configureverwendetwidget.source.maximum/minimumals Felder, ruft sie dann aber als Methoden auf. Dadurch ist die Bedingung immer false, falls diese nur als Methoden verfügbar sind. Überlege, stattdessen aufwidget.source.maximum ~= nilzu prüfen oder durchgängig die Methoden‑API zu verwenden. - Im
else‑Zweig derconfigure‑Funktion ist die Reihenfolge der ZahlenfelderthresholdUpundthresholdDownim Vergleich zum Original (Down dann Up) vertauscht, was unbeabsichtigt die Bindings vertauschen und Nutzer verwirren kann. Richte die Reihenfolge an der bestehenden UI aus. - Die
menu‑Funktion führt einen hart codierten KategorienwertCATEGORY_LUA = 29ein. Wenn möglich, verwende bestehende Kategorie‑Konstanten wieder oder zentralisiere diese Werte, um Magic Numbers zu vermeiden und das Kategorie‑Handling über Widgets hinweg konsistent zu halten.
Prompt für KI‑Agenten
Bitte bearbeite die Kommentare aus diesem Code‑Review:
## Allgemeine Kommentare
- Die dynamische Threshold‑Bereichsprüfung in `configure` verwendet `widget.source.maximum`/`minimum` als Felder, ruft sie dann aber als Methoden auf. Dadurch ist die Bedingung immer false, falls diese nur als Methoden verfügbar sind. Überlege, stattdessen auf `widget.source.maximum ~= nil` zu prüfen oder durchgängig die Methoden‑API zu verwenden.
- Im `else`‑Zweig der `configure`‑Funktion ist die Reihenfolge der Zahlenfelder `thresholdUp` und `thresholdDown` im Vergleich zum Original (Down dann Up) vertauscht, was unbeabsichtigt die Bindings vertauschen und Nutzer verwirren kann. Richte die Reihenfolge an der bestehenden UI aus.
- Die `menu`‑Funktion führt einen hart codierten Kategorienwert `CATEGORY_LUA = 29` ein. Wenn möglich, verwende bestehende Kategorie‑Konstanten wieder oder zentralisiere diese Werte, um Magic Numbers zu vermeiden und das Kategorie‑Handling über Widgets hinweg konsistent zu halten.
## Einzelne Kommentare
### Kommentar 1
<location> `main.lua:363-372` </location>
<code_context>
-- thresholds
- wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
- wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
+ if wHelper.existSource(widget.source) and widget.source.maximum and widget.source.minimum then
+ -- Not perfect since to have this code working you will need
+ -- to configure the widget to set the source first
+ -- and then to configure again to set the remaining parameters
+ wConfig.addNumberField("thresholdDown", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
+ wConfig.addNumberField("thresholdUp", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
+ else
+ wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
+ wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
+ end
</code_context>
<issue_to_address>
**suggestion:** Threshold‑Felder werden je nach Branch unterschiedlich angeordnet, was zu einem inkonsistenten UI‑Layout führen kann.
Im `if`‑Zweig werden die Felder als `thresholdDown` und dann `thresholdUp` hinzugefügt, im `else`‑Zweig jedoch in umgekehrter Reihenfolge. Wenn die Konfig‑UI die Deklarationsreihenfolge berücksichtigt, unterscheidet sich dadurch das Layout je nachdem, ob eine gültige Quelle existiert. Bitte richte die Reihenfolge in beiden Zweigen aneinander aus.
```suggestion
if wHelper.existSource(widget.source) and widget.source.maximum and widget.source.minimum then
-- Not perfect since to have this code working you will need
-- to configure the widget to set the source first
-- and then to configure again to set the remaining parameters
wConfig.addNumberField("thresholdDown", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdUp", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
else
wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
end
```
</issue_to_address>
### Kommentar 2
<location> `lib/w_paint.lua:147` </location>
<code_context>
footerHeight = wPaint.FREE_ABOVE + footerHeight
lcd.color(footerBGColor)
- lcd.drawFilledRectangle(0, widget.height - footerHeight, widget.width, widget.height)
+ lcd.drawFilledRectangle(wPaint.TRANSPARENCY_X_OFFSET, widget.height - footerHeight - (2 * wPaint.TRANSPARENCY_Y_OFFSET), widget.width - (2 * wPaint.TRANSPARENCY_X_OFFSET), widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET))
end
--- draw footer text
</code_context>
<issue_to_address>
**issue (bug_risk):** Die Höhe des Footer‑Rechtecks wirkt inkonsistent mit dem vorgesehenen Footer‑Band und mit der Verwendung der Transparenz‑Offsets für Titel und Hintergrund.
Beim Titel und beim Haupt‑Hintergrund entspricht das `height`‑Argument jeweils nur dem zu zeichnenden Bereich (z. B. `widget.height - (2 * TRANSPARENCY_Y_OFFSET)` für den Hintergrund, `titleHeight - TRANSPARENCY_Y_OFFSET` für den Titel). Hier wird `y` des Footers auf `widget.height - footerHeight - 2 * TRANSPARENCY_Y_OFFSET` verschoben, aber `height` ist `widget.height - 2 * TRANSPARENCY_Y_OFFSET`, sodass das Rechteck weit oberhalb des Footer‑Bands reicht. Verwende besser `footerHeight` (oder `footerHeight - TRANSPARENCY_Y_OFFSET`) für die Footer‑Höhe, damit das Rechteck auf den Footer‑Bereich innerhalb der Transparenzbox begrenzt ist.
</issue_to_address>Sourcery ist für Open Source kostenlos – wenn dir unsere Reviews gefallen, freuen wir uns, wenn du sie teilst ✨
Original comment in English
Hey - I've found 2 issues, and left some high level feedback:
- The dynamic threshold range check in
configureuseswidget.source.maximum/minimumas fields but then calls them as methods, so the condition will always be false if these are only exposed as methods; consider checking viawidget.source.maximum ~= nilor directly using the method API consistently. - In the
configurefunction’selsebranch, the order ofthresholdUpandthresholdDownnumber fields is reversed compared to the original (Down then Up), which may unintentionally swap the bindings and confuse users; align the order with the existing UI. - The
menufunction introduces a hard‑coded category valueCATEGORY_LUA = 29; if possible, reuse existing category constants or centralize these values to avoid magic numbers and keep category handling consistent across widgets.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The dynamic threshold range check in `configure` uses `widget.source.maximum`/`minimum` as fields but then calls them as methods, so the condition will always be false if these are only exposed as methods; consider checking via `widget.source.maximum ~= nil` or directly using the method API consistently.
- In the `configure` function’s `else` branch, the order of `thresholdUp` and `thresholdDown` number fields is reversed compared to the original (Down then Up), which may unintentionally swap the bindings and confuse users; align the order with the existing UI.
- The `menu` function introduces a hard‑coded category value `CATEGORY_LUA = 29`; if possible, reuse existing category constants or centralize these values to avoid magic numbers and keep category handling consistent across widgets.
## Individual Comments
### Comment 1
<location> `main.lua:363-372` </location>
<code_context>
-- thresholds
- wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
- wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
+ if wHelper.existSource(widget.source) and widget.source.maximum and widget.source.minimum then
+ -- Not perfect since to have this code working you will need
+ -- to configure the widget to set the source first
+ -- and then to configure again to set the remaining parameters
+ wConfig.addNumberField("thresholdDown", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
+ wConfig.addNumberField("thresholdUp", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
+ else
+ wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
+ wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
+ end
</code_context>
<issue_to_address>
**suggestion:** Threshold fields are added in different order depending on the source branch, which may lead to inconsistent UI layout.
In the `if` branch the fields are added as `thresholdDown` then `thresholdUp`, but in the `else` branch they are added in the opposite order. If the config UI respects declaration order, this will cause the layout to differ depending on whether a valid source exists. Please align the order in both branches.
```suggestion
if wHelper.existSource(widget.source) and widget.source.maximum and widget.source.minimum then
-- Not perfect since to have this code working you will need
-- to configure the widget to set the source first
-- and then to configure again to set the remaining parameters
wConfig.addNumberField("thresholdDown", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdUp", widget.source:minimum(), widget.source:maximum(), THRESHOLD_PRECISION)
else
wConfig.addNumberField("thresholdDown", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
wConfig.addNumberField("thresholdUp", -THRESHOLD_RANGE, THRESHOLD_RANGE, THRESHOLD_PRECISION)
end
```
</issue_to_address>
### Comment 2
<location> `lib/w_paint.lua:147` </location>
<code_context>
footerHeight = wPaint.FREE_ABOVE + footerHeight
lcd.color(footerBGColor)
- lcd.drawFilledRectangle(0, widget.height - footerHeight, widget.width, widget.height)
+ lcd.drawFilledRectangle(wPaint.TRANSPARENCY_X_OFFSET, widget.height - footerHeight - (2 * wPaint.TRANSPARENCY_Y_OFFSET), widget.width - (2 * wPaint.TRANSPARENCY_X_OFFSET), widget.height - (2 * wPaint.TRANSPARENCY_Y_OFFSET))
end
--- draw footer text
</code_context>
<issue_to_address>
**issue (bug_risk):** Footer rectangle height looks inconsistent with the intended footer band and with the title/background usage of transparency offsets.
For the title and main background, the `height` argument matches only the region being drawn (e.g., `widget.height - (2 * TRANSPARENCY_Y_OFFSET)` for the background, `titleHeight - TRANSPARENCY_Y_OFFSET` for the title). Here, the footer `y` is shifted by `widget.height - footerHeight - 2 * TRANSPARENCY_Y_OFFSET`, but `height` is `widget.height - 2 * TRANSPARENCY_Y_OFFSET`, so the rectangle extends well above the footer band. Consider using `footerHeight` (or `footerHeight - TRANSPARENCY_Y_OFFSET`) for the footer height so the rectangle is limited to the footer area within the transparency box.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
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.
Hello Andreas,
I suggest you include the following to improve 3stated:
As I'm coding on a Mac, I also edited the gitignore file.
Regards,
Eric Delord
Zusammenfassung von Sourcery
Fügt dem 3‑Zustands-Widget einen visuellen Fokusabstand, einen Kontextmenüeintrag zum Zurücksetzen der Quelle und dynamische Schwellwertbereiche hinzu, zusammen mit Aktualisierungen der Lokalisierung und der Repository-Ignorierliste.
Neue Funktionen:
Verbesserungen:
Build:
.gitignore-Einträge für das Repository.Original summary in English
Summary by Sourcery
Add visual focus padding, a source-reset context menu entry, and dynamic threshold ranges to the 3-state widget, along with localization and repo ignore updates.
New Features:
Enhancements:
Build:
Neue Funktionen:
Verbesserungen:
Build:
Original summary in English
Zusammenfassung von Sourcery
Fügt dem 3‑Zustands-Widget einen visuellen Fokusabstand, einen Kontextmenüeintrag zum Zurücksetzen der Quelle und dynamische Schwellwertbereiche hinzu, zusammen mit Aktualisierungen der Lokalisierung und der Repository-Ignorierliste.
Neue Funktionen:
Verbesserungen:
Build:
.gitignore-Einträge für das Repository.Original summary in English
Summary by Sourcery
Add visual focus padding, a source-reset context menu entry, and dynamic threshold ranges to the 3-state widget, along with localization and repo ignore updates.
New Features:
Enhancements:
Build: