fix(kronolith): group sidebar share icon with action controls#62
Merged
Conversation
Move the shared calendar/tasklist indicator out of the resource label and float it with refresh/edit on the right. Constrain the attendees graphic size after the Lucide asset bump.
ralflang
added a commit
that referenced
this pull request
Jun 30, 2026
Release version 5.0.0 test: add @coversNothing to coverage-neutral test style: explicit nullable type and corrected block indentation chore: update CI feat(kronolith): simplify iTIP invitation mail and templates Merge pull request #62 from horde/fix/icon-sidebar-ac fix(kronolith): group sidebar share icon with action controls Merge pull request #61 from horde/feat/disallow-new-time-proposal-ui Potential fix for pull request finding feat(kronolith): organizer UI for disallow new time proposals fix(block): close _eventsForDay method in portal Month block Merge pull request #59 from horde/feat/polish_EAS16.1 fix(kronolith): harden EAS 16.1 proposal-clear export Merge pull request #60 from horde/fix/portal-month-block Potential fix for pull request finding fix(kronolith): show events in This Month portal block feat(kronolith): EAS 16.1 counter-proposals and proposal clear on sync fix(kronolith): scope ESC navigation and restore RedBox fade Delete doc/pr directory fix(kronolith): allow reopening event dialog after close fix(kronolith): guard count() on null cancellations in sendITipNotifications fix(kronolith): preserve timezone in attendee proposal storage Potential fix for pull request finding Potential fix for pull request finding Potential fix for pull request finding feat(kronolith): store and sync EAS 16.1 attendee proposals and disallow flags Refactor oldProperties assignment using null coalescing Merge pull request #56 from horde/fix/add_event_EAS16.0_allday Merge branch 'FRAMEWORK_6_0' into fix/add_event_EAS16.0_allday fix(maps): use sensible default zoom when opening event map Merge pull request #55 from horde/fix/icons_II Fix CSS selectors for iconEdit/iconDelete to target anchor-wrapped images fix(icon): deduplicate code feat(kronolith): implement EAS 16.0 calendar sync and all-day import fix fix(kronolith): apply EAS 16 date-only import for new all-day events fix(kronolith): use transparent 48px event icons with background-aware colors
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
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.
Summary
Motivation
After the icon modernization, the share icon rendered too large inline with wrapped resource names while edit/refresh stayed on the right edge.
Changes
addShareIcon()out of the label span and into the row action group.horde-resource-share-*and tighten action icon spacing<img>sizing rules for event/status viewsTest plan