Skip to content

Conversation

smp46
Copy link
Owner

@smp46 smp46 commented Sep 17, 2025

Continuation of un-merged PR stonith404/pingvin-share/pull/813.

Description:

This PR enhances UI clarity by adding descriptive text labels to header icons and implementing tooltips for the main action icon buttons.
Motivation:

The goal is to improve usability and reduce potential confusion for all users. Explicit text labels in the header section make the function of those elements immediately clear. For the main action buttons (like Info, Undo, etc.), tooltips were added to clarify icon function upon hover, providing necessary context without permanently adding text to that potentially dense button area.
Changes:

  • Added descriptive text labels next to icons in the header section.
  • Implemented descriptive tooltips for main action icon buttons (Info, Undo, Download, Copy, etc.). (Changed from original plan of adding text labels here).
  • Included corresponding translations for the new header labels and action button tooltip text.
  • Code has been formatted using npm run format.

@smp46
Copy link
Owner Author

smp46 commented Sep 18, 2025

Updates have been made in line with the original suggested changes made by stonish404:

  • Authenticated Header "Shares" and "Profile" buttons have been restored to their original icon, but with a tooltip on hover.
  • The "Download All" button has been restored to purple, and the copy link button has been made orange.

The "HoverTip" component now wraps all (new) usages of Tooltip, with width?, label, children, disabled? as props. position="bottom", multine, events... are now predefined in the HoverTip component. disabled was added so the HoverTip would disappear after clicking on a menu icon.

@smp46 smp46 merged commit 2510a8a into main Sep 18, 2025
2 checks passed
@smp46 smp46 deleted the add-descriptive-text-to-icons branch September 18, 2025 03:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants