Skip to content

Conversation

@Magnusrm
Copy link
Contributor

@Magnusrm Magnusrm commented Nov 11, 2025

Description

Remove redundant css causing bug for datepicker while in repeating group.

Related Issue(s)

Verification/QA

  • Manual functionality testing
    • I have tested these changes manually
    • Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
    • No testing done/necessary
  • Automated tests
    • Unit test(s) have been added/updated
    • Cypress E2E test(s) have been added/updated
    • No automatic tests are needed here (no functional changes/additions)
    • I want someone to help me make some tests
  • UU/WCAG (follow these guidelines until we have our own)
    • I have tested with a screen reader/keyboard navigation/automated wcag validator
    • No testing done/necessary (no DOM/visual changes)
    • I want someone to help me perform accessibility testing
  • User documentation @ altinn-studio-docs
    • Has been added/updated
    • No functionality has been changed/added, so no documentation is needed
    • I will do that later/have created an issue
  • Support in Altinn Studio
    • Issue(s) created for support in Studio
    • This change/feature does not require any changes to Altinn Studio
  • Sprint board
    • The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
    • I don't have permissions to do that, please help me out
  • Labels
    • I have added a kind/* and backport* label to this PR for proper release notes grouping
    • I don't have permissions to add labels, please help me out

Summary by CodeRabbit

Release Notes

  • Style
    • Enhanced table row hover effects in repeating group tables to display the default visual feedback.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 11, 2025

📝 Walkthrough

Walkthrough

A CSS hover style rule was removed from the RepeatingGroup table module. The change eliminates an explicit background-color: unset declaration on hover for table body rows, allowing hover states to default to inherited or browser-default styling instead of being explicitly reset.

Changes

Cohort / File(s) Change Summary
CSS styling update
src/layout/RepeatingGroup/RepeatingGroup.module.css
Removed background-color: unset from .repeatingGroupTable tbody hover state, allowing default hover styles to apply

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Remove unnecessary css causing bug' directly relates to the main change: removing redundant CSS that caused a bug with the datepicker in repeating groups.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed PR description includes Related Issue, manual testing verification, WCAG testing, and confirms no Studio changes needed, sprint board updated, and labels added.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/datepicker-styles-removed-on-hover

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Magnusrm Magnusrm changed the title remove unnecessary css causing bug Remove unnecessary css causing bug Nov 11, 2025
@Magnusrm Magnusrm added kind/bug Something isn't working backport-ignore This PR is a new feature and should not be cherry-picked onto release branches labels Nov 11, 2025
@sonarqubecloud
Copy link

@Magnusrm Magnusrm merged commit 41959ab into main Nov 12, 2025
16 of 18 checks passed
@Magnusrm Magnusrm deleted the fix/datepicker-styles-removed-on-hover branch November 12, 2025 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-ignore This PR is a new feature and should not be cherry-picked onto release branches kind/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Styling issue with datepicker on hover when in repeating group

2 participants