Skip to content

Fix (input-file) unify cursor styling and prevent focus when disabled#7619

Merged
deleonio merged 2 commits intopublic-ui:developfrom
aTTiCuZ-CodingPage:7572-fix-cursor-styling-input-file
May 6, 2025
Merged

Fix (input-file) unify cursor styling and prevent focus when disabled#7619
deleonio merged 2 commits intopublic-ui:developfrom
aTTiCuZ-CodingPage:7572-fix-cursor-styling-input-file

Conversation

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor

@aTTiCuZ-CodingPage aTTiCuZ-CodingPage commented Apr 30, 2025

The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer:

  • Meaningful pull request title for the release notes
  • Pull request is linked to an issue and all changes relate to the issue
  • Tests to protect this code implemented (if applicable)
  • Manual test performed successfully (if applicable)
  • Documentation or migration has been updated (if applicable)

@publicuibot
Copy link
Copy Markdown
Contributor

publicuibot bot commented Apr 30, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@anicyne anicyne requested a review from sdvg April 30, 2025 08:36
@deleonio
Copy link
Copy Markdown
Contributor

Hi @sdvg,

du warst ja an den Cursorn dran - kannst du das kurz reviewn.

Wenn ja, dann auch Cherry-Picken für v2.

Viele Grüße
Martin

@deleonio
Copy link
Copy Markdown
Contributor

deleonio commented May 5, 2025

Hi @aTTiCuZ-CodingPage,

kannst Du bitte hierzu ein Ticket anlegen, was der Änderungsgrund beschreibt: https://github.com/public-ui/kolibri/issues/new?template=BLANK_ISSUE

Vielen Dank

@anicyne anicyne linked an issue May 5, 2025 that may be closed by this pull request
@sdvg
Copy link
Copy Markdown
Contributor

sdvg commented May 5, 2025

@aTTiCuZ-CodingPage

Vielen Dank!

Lokaler Test: Sieht gut aus in Chrome, Firefox und Safari 👍

  • Im Ticket ist noch ein zweites Problem geschrieben (in den Kommentaren). Würdest du das gerne auch gleich noch anschauen, oder sollten wir es lieber in ein neues Ticket verschieben?
  • Mit dem CLA hat noch etwas nicht geklappt, siehe Kommentar oben.
  • Die Änderungen beziehen sich auf den develop-Branch und passen soweit für mich. Wir bräuchten die selben Änderungen auch noch für release/2.

@sdvg
Copy link
Copy Markdown
Contributor

sdvg commented May 5, 2025

Der DoD-Check wird bei externen Pull Requests nicht funktionieren.
@aTTiCuZ-CodingPage Könntest du die Checkliste vielleicht händisch in die PR-Beschreibung kopieren, oder zumindest gegenchecken?

https://github.com/public-ui/kolibri/blob/develop/.github/dod.yml

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor Author

aTTiCuZ-CodingPage commented May 5, 2025

@sdvg

  • Das zweite Problem, was das Fokussieren, wenn die Komponente disabled ist, angeht. Das habe ich auch behoben.
  • Ich schaue mir das CLA Problem an
  • OK, ich baue die Änderung auch für release/2 ein

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor Author

recheck

@deleonio deleonio added the v3 label May 6, 2025
deleonio

This comment was marked as outdated.

@deleonio deleonio self-requested a review May 6, 2025 05:13
Copy link
Copy Markdown
Contributor

@deleonio deleonio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bitte immer erst release/2 vor develop umsetzen und Review von release/2 abwarten.

FYI: @aTTiCuZ-CodingPage, @sdvg, @anicyne, @laske185

@deleonio deleonio merged commit bfd616e into public-ui:develop May 6, 2025
8 of 9 checks passed
@publicuibot publicuibot bot locked and limited conversation to collaborators May 6, 2025
@deleonio deleonio removed the v3 label Jun 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Styling Probleme mit KolInputFile

3 participants