Skip to content

fix: title is taking to much space in the titlebar#101

Merged
perber merged 4 commits into
mainfrom
fix/95-title-is-taking-to-much-space-in-the-titlebar
Jun 17, 2025
Merged

fix: title is taking to much space in the titlebar#101
perber merged 4 commits into
mainfrom
fix/95-title-is-taking-to-much-space-in-the-titlebar

Conversation

@perber
Copy link
Copy Markdown
Owner

@perber perber commented Jun 17, 2025

adjusting css to truncate title

Copilot AI review requested due to automatic review settings June 17, 2025 21:28
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the EditorTitleBar by truncating long titles with a tooltip fallback and adjusting the "dirty" indicator behavior on mobile devices.

  • Introduces useIsMobile to switch the dirty indicator between text and an asterisk on small screens
  • Wraps the title (and related elements) in TooltipWrapper and applies CSS truncation
  • Adds responsive truncation to the slug display
Comments suppressed due to low confidence (1)

ui/leafwiki-ui/src/components/EditorTitleBar.tsx:31

  • New conditional rendering for mobile vs. desktop dirty indicators should be covered by unit or integration tests to verify both variants show correctly.
          {isDirty && !isMobile && (

Comment thread ui/leafwiki-ui/src/components/EditorTitleBar.tsx Outdated
Comment thread ui/leafwiki-ui/src/components/EditorTitleBar.tsx Outdated
Comment thread ui/leafwiki-ui/src/components/EditorTitleBar.tsx
Comment thread ui/leafwiki-ui/src/components/EditorTitleBar.tsx
@perber perber linked an issue Jun 17, 2025 that may be closed by this pull request
perber and others added 3 commits June 17, 2025 23:31
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@perber perber merged commit 87037a7 into main Jun 17, 2025
4 checks passed
@perber perber deleted the fix/95-title-is-taking-to-much-space-in-the-titlebar branch June 17, 2025 22:06
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.

On the mobile phone the title is taking to much space

2 participants