Skip to content

Agentic UI: Make the sidebar footer a direct Settings entry - #4354

Merged
nightnei merged 2 commits into
trunkfrom
claude/sidebar-footer-settings-913110
Jul 28, 2026
Merged

Agentic UI: Make the sidebar footer a direct Settings entry#4354
nightnei merged 2 commits into
trunkfrom
claude/sidebar-footer-settings-913110

Conversation

@nightnei

@nightnei nightnei commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Related issues

How AI was used in this PR

AI implemented the change following the direction of #4311, I reviewed and tested

Proposed Changes

When you click on Avatar in left bottom corner we should land users on Settigns page instead of dropdown.
We do it to unify behavior for logged-in and logged-out users, and all links in the dropdown are already available on the Settings page.
See testing instructions.

Testing Instructions

  1. Run Agentic UI
  2. Sign in - assert that you see your avatar and Settings label. Clicking on it should lead you directly to the settings page
BEFORE AFTER
Screenshot 2026-07-27 at 12 40 36 Screenshot 2026-07-27 at 12 37 48
3. Log out 4. Assert that now you see the Settings icon insteadof avatar and click on it still leads you to settings page
BEFORE AFTER
Screenshot 2026-07-27 at 12 40 14 Screenshot 2026-07-27 at 12 38 06
5. Open Settings page and assert that you see tooltips for "Docs" and for "Report an issue"
Screenshot 2026-07-27 at 14 47 37

@nightnei nightnei self-assigned this Jul 27, 2026
@wpmobilebot

wpmobilebot commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

📊 Performance Test Results

Comparing 7c00029 vs trunk

app-size

Metric trunk 7c00029 Diff Change
App Size (Mac) 1375.46 MB 1381.97 MB +6.51 MB 🔴 0.5%

site-editor

Metric trunk 7c00029 Diff Change
load 1046 ms 1078 ms +32 ms ⚪ 0.0%

site-startup

Metric trunk 7c00029 Diff Change
siteCreation 7022 ms 6968 ms 54 ms 🟢 -0.8%
siteStartup 2350 ms 2351 ms +1 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@nightnei
nightnei requested review from bcotrim and sejas July 27, 2026 13:50

@bcotrim bcotrim left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Small suggestions:

  1. There is no longer a link to the WordPress.com profile. Maybe intentional, but we had that link the menu. Maybe we could include it on the app settings account section?
  2. Log in button in app settings is still available when offline (maybe pre-existing and it doesn't hurt since the browser will fail to open the page). It's a simple change so I would include it
Image

@nightnei

Copy link
Copy Markdown
Contributor Author

LGTM

Small suggestions:

  1. There is no longer a link to the WordPress.com profile. Maybe intentional, but we had that link the menu. Maybe we could include it on the app settings account section?
  2. Log in button in app settings is still available when offline (maybe pre-existing and it doesn't hurt since the browser will fail to open the page). It's a simple change so I would include it
Image

Both are addressed
Screenshot 2026-07-27 at 18 25 31

@nightnei
nightnei merged commit 28e479c into trunk Jul 28, 2026
13 checks passed
@nightnei
nightnei deleted the claude/sidebar-footer-settings-913110 branch July 28, 2026 10:13
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.

3 participants