Skip to content

feat(desktop): Prioritize suffix when truncating path in header #3110

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
lily-de merged 1 commit into from
Jul 1, 2025
Merged

feat(desktop): Prioritize suffix when truncating path in header #3110

lily-de merged 1 commit into from
Jul 1, 2025

Conversation

@exitcode0
Copy link
Contributor

@exitcode0 exitcode0 commented Jun 27, 2025

Summary

For long file paths, the last segments (e.g. the current directory) are typically the most relevant.
We now truncate from the start of the path instead of the end, so users see the meaningful suffix rather than a generic unchanging prefix like /Users/foo/.

Old Behavior

  • Path Field was a fixed size
  • If the path did not fit, Truncated the most useful information from the end of the string

New Behavior

  • Path field grows to fit the path if there is room to do so
  • If the path still does not fit, Truncate from the beginning of the string

Changes

  • RTL truncation (direction: rtl)
    • Makes text truncate from the left, keeping the rightmost (suffix) part visible.
  • unicode-bidi: plaintext
    • Prevents path segments from rendering out of order when using RTL.
  • Switched <div> to <span>
    • Inline behavior avoids unnecessary layout constraints.
  • Added flex-grow + min-width: 0
    • Ensures the path container shrinks and grows properly in flex layouts.
  • Applied overflow-hidden + text-ellipsis
    • Enables clean truncation without layout overflow.

@exitcode0
feat(desktop): working directory field in header now flex-grows and t…
e25072c 
…runcates from the start

- The working directory field in the header now expands to fill available space (flex-grow)
- If the path is too long, it truncates from the start, so the most relevant (ending) part is always visible
- Tooltip still shows the full path on hover
- Improves UX for users with long directory paths
@exitcode0 exitcode0 force-pushed the header-path-truncate branch from 4b344f3 to e25072c Compare June 30, 2025 21:32
@lily-de lily-de merged commit e49985e into block:main Jul 1, 2025
6 checks passed
jsibbison-square added a commit that referenced this pull request Jul 1, 2025
@jsibbison-square
Merge remote-tracking branch 'origin/main' into jsibbison-20250630-st…
7373b5e 
…ructured-output

* origin/main: (22 commits)
  feat(desktop): Prioritize suffix when truncating path in header (#3110)
  chore(release): release version 1.0.31 (#3185)
  feat: additional sub recipes via command line (#3163)
  Add Internal Recipes To Recipes Cookbook (#3179)
  pipe the argument to storage (#3184)
  docs: removing comment (#3183)
  docs: add generator option to create recipe (#3182)
  update the path for temporal (#3131)
  docs: add link to Square MCP (#3181)
  attempt to fix build #3 (#3180)
  attempt  fix folde permissions for windows build (#3178)
  attempt to fix windows cli permission issue (#3177)
  allow to use dev/null for no-session mode (#3176)
  feat: change naming of recipe creation in more menu links (#3175)
  Docs: Add Recipe video to landing page (#3173)
  Docs: Create new directory when starting new session (#3174)
  fixes cron parsing issues (#3172)
  fix: handle Windows package subdirectory in CLI installation script (#3171)
  fixed the npx/uvx content (#3170)
  Mark helper scripts as executable (#3169)
  ...
AaronGoldsmith added a commit to AaronGoldsmith/goose that referenced this pull request Jul 2, 2025
@AaronGoldsmith
Merge branch 'main' into ag/feat-desktop-recipe-params
04bf72c 
* main: (37 commits)
  fix: fix desktop recipe url generation (block#3209)
  feat: improve UX for saving recipes (block#3214)
  fix: Pass Google AI API key in HTTP header, not query param (block#3192)
  docs: add linter to CONTRIBUTING.md (block#3168)
  feat: Structured output for recipes (block#3188)
  Fix cost tracking accuracy and OpenRouter model pricing (block#3189)
  docs: update cli install instructions for windows (block#3205)
  Docs: Cost tracking on the desktop app (block#3204)
  feat: Adding streamable-http transport support for backend, desktop and cli (block#2942)
  fix: use the correct `contains` syntax on create-recipe-pr.yml (block#3193)
  Temporarily Remove GH Copilot Provider  (block#3199)
  docs: fix tab navigation (block#3201)
  feat: use tiktoken-rs instead of tokenizers, single global tokenizer (block#3115)
  add playwright-mcp server to extensions list (block#3010)
  Add `/extension` path for extension installation (block#3011)
  feat(desktop): Prioritize suffix when truncating path in header (block#3110)
  chore(release): release version 1.0.31 (block#3185)
  feat: additional sub recipes via command line (block#3163)
  Add Internal Recipes To Recipes Cookbook (block#3179)
  pipe the argument to storage (block#3184)
  ...
@chaitanyarahalkar chaitanyarahalkar mentioned this pull request Jul 5, 2025
Open
atarantino pushed a commit to atarantino/goose that referenced this pull request Jul 14, 2025
@exitcode0 @atarantino
feat(desktop): Prioritize suffix when truncating path in header (bloc…
1b75979 
…k#3110)

Signed-off-by: Adam Tarantino <[email protected]>
@exitcode0 exitcode0 deleted the header-path-truncate branch July 17, 2025 04:38
s-soroosh pushed a commit to s-soroosh/goose that referenced this pull request Jul 18, 2025
@exitcode0 @s-soroosh
feat(desktop): Prioritize suffix when truncating path in header (bloc…
ae059de 
…k#3110)

Signed-off-by: Soroosh <[email protected]>
cbruyndoncx pushed a commit to cbruyndoncx/goose that referenced this pull request Jul 20, 2025
@exitcode0
feat(desktop): Prioritize suffix when truncating path in header (bloc…
9009194 
…k#3110)
pedronveloso added a commit to pedronveloso/goose that referenced this pull request Jul 28, 2025
@pedronveloso
Merge branch 'main' into pedro/settings-back-navigation
997b213 
* main: (22 commits)
  add playwright-mcp server to extensions list (block#3010)
  Add `/extension` path for extension installation (block#3011)
  feat(desktop): Prioritize suffix when truncating path in header (block#3110)
  chore(release): release version 1.0.31 (block#3185)
  feat: additional sub recipes via command line (block#3163)
  Add Internal Recipes To Recipes Cookbook (block#3179)
  pipe the argument to storage (block#3184)
  docs: removing comment (block#3183)
  docs: add generator option to create recipe (block#3182)
  update the path for temporal (block#3131)
  docs: add link to Square MCP (block#3181)
  attempt to fix build block#3 (block#3180)
  attempt  fix folde permissions for windows build (block#3178)
  attempt to fix windows cli permission issue (block#3177)
  allow to use dev/null for no-session mode (block#3176)
  feat: change naming of recipe creation in more menu links (block#3175)
  Docs: Add Recipe video to landing page (block#3173)
  Docs: Create new directory when starting new session (block#3174)
  fixes cron parsing issues (block#3172)
  fix: handle Windows package subdirectory in CLI installation script (block#3171)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lily-de lily-de lily-de approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@exitcode0 @lily-de