Skip to content

feat: tooltip revamped (refs SFKUI-6500)#86

Merged
ext merged 2 commits intomainfrom
feature/revamped-tooltip
Nov 28, 2024
Merged

feat: tooltip revamped (refs SFKUI-6500)#86
ext merged 2 commits intomainfrom
feature/revamped-tooltip

Conversation

@ext
Copy link
Contributor

@ext ext commented Oct 23, 2024 •
edited
Loading

image

image

Om ni provkör se till att ställa in att animationer ska vara aktiverade, den nya koden respekterar prefers-reduced-motion och det är förinställt till reduce på fjärrskrivborden.

Todo:

  • Rätt jira
  • Städa bort oanvänd kod
  • Städa upp exempel
  • Kompletera med testfall?
  • Screenshot tester?
  • Grafikavstämning
  • Återställ stäng-knapp och annat visuellt

@ext ext requested a review from mitra-Re October 23, 2024 16:33
@ext
Copy link
Contributor Author

ext commented Oct 23, 2024

@mitra-Re Vill du börja regressionstesta denna?

@github-actions
Copy link

github-actions bot commented Oct 23, 2024 •
edited
Loading

PR Preview Action v1.4.8
Preview removed because the pull request was closed.
2024-11-28 07:34 UTC

@ext ext force-pushed the feature/revamped-tooltip branch 5 times, most recently from b0258ce to 65f57af Compare October 25, 2024 11:39
dvitamin
dvitamin previously approved these changes Oct 28, 2024
View reviewed changes
@TMFKse
Copy link
Contributor

TMFKse commented Oct 28, 2024

Kan vi göra något för att tvinga semantisk rubriksnivå ifall #header-slotten används? Det är ju lite problematiskt att den defaultar till div.

@ext
Copy link
Contributor Author

ext commented Oct 28, 2024

Kan vi göra något för att tvinga semantisk rubriksnivå ifall #header-slotten används? Det är ju lite problematiskt att den defaultar till div.

@TMFKse Vad är bäst isf tror du? Tvinga att man sätter headerTag propen? Eller ska vi gissa att det är rimligt med h3?

@TMFKse
Copy link
Contributor

TMFKse commented Oct 28, 2024

Kan vi göra något för att tvinga semantisk rubriksnivå ifall #header-slotten används? Det är ju lite problematiskt att den defaultar till div.

@TMFKse Vad är bäst isf tror du? Tvinga att man sätter headerTag propen? Eller ska vi gissa att det är rimligt med h3?

Jag tänker egentligen att det klokaste vore att tvinga headerTag-prop i de fall slotten används. Lite osäker på om/hur man skulle göra det dock? Ifall det inte går att lösa, vore det väl bättre att defaulta till h3, ja. Det känns dumt att tvinga konsumenten att sätta en headerTag samtidigt som det nästan är rekommenderat att inte använda rubriker, utan bara "brödtext".

@ext ext force-pushed the feature/revamped-tooltip branch from 65f57af to 642890b Compare October 29, 2024 07:23
@ext ext force-pushed the feature/revamped-tooltip branch 2 times, most recently from 4906177 to 9d69fd4 Compare November 5, 2024 15:06
NicklasWei
NicklasWei previously approved these changes Nov 6, 2024
View reviewed changes
MCFK
MCFK previously approved these changes Nov 6, 2024
View reviewed changes
@ext ext dismissed stale reviews from Marie-Jakobsson, MCFK, and NicklasWei via bf92765 November 27, 2024 23:57
@ext ext force-pushed the feature/revamped-tooltip branch 2 times, most recently from bf92765 to 7a7c9b9 Compare November 28, 2024 00:06
@ext ext force-pushed the feature/revamped-tooltip branch 2 times, most recently from 98bee99 to 64e3e51 Compare November 28, 2024 06:22
MCFK
MCFK reviewed Nov 28, 2024
View reviewed changes
MCFK
MCFK previously approved these changes Nov 28, 2024
View reviewed changes
@ext ext force-pushed the feature/revamped-tooltip branch from 64e3e51 to ef8d2f2 Compare November 28, 2024 07:01
@ext ext merged commit 082d1f8 into main Nov 28, 2024
@ext ext deleted the feature/revamped-tooltip branch December 6, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MCFK MCFK MCFK approved these changes

@NicklasWei NicklasWei NicklasWei left review comments

@dvitamin dvitamin dvitamin left review comments

@Marie-Jakobsson Marie-Jakobsson Marie-Jakobsson left review comments

@kristofernilsson kristofernilsson Awaiting requested review from kristofernilsson

@kristinalindstrom kristinalindstrom Awaiting requested review from kristinalindstrom kristinalindstrom is a code owner

@OlofFredriksson OlofFredriksson Awaiting requested review from OlofFredriksson

@TMFKse TMFKse Awaiting requested review from TMFKse TMFKse is a code owner

@mitra-Re mitra-Re Awaiting requested review from mitra-Re

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@ext @TMFKse @NicklasWei @dvitamin @MCFK @Marie-Jakobsson