Skip to content

docs: MFS delegation UX mocks for #802 - #837

Open
thebeyondr wants to merge 23 commits into
mainfrom
802-design-mfs-delegation-ux
Open

thebeyondr wants to merge 23 commits into
mainfrom
802-design-mfs-delegation-ux

Conversation

@thebeyondr

@thebeyondr thebeyondr commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

For #802

Design mocks only (no product UI yet) for MFS / fee-vs-stake delegation UX.

Summary

  • Adds design/mocks/mfs-delegation-802/ (list, mobile market cards, orchestrator detail, delegation status, day-one launch checklist)
  • Shares self-hosted Inter under design/mocks/shared/ for HTML previews
  • Captures MFS UI rules in AGENTS.md

Screenshots

image image image

Test plan

  • Open design/mocks/mfs-delegation-802/index.html and confirm Inter loads
  • Skim list / mobile / detail / day-one for fee=blue, stake=violet, binding-only tint
  • Confirm day-one uses brand green + Livepeer mark, with gateway → fees → stake steps
  • Open design/mocks/governance-voting/index.html and confirm shared Inter still resolves

Stakeholder-ready mockups for #802 and durable UI guidance so future work matches the agreed MFS delegation patterns.
Vendored latin 400–700 woff2 from @fontsource/inter so the mock loads on file:// without Google Fonts or node_modules paths.
Keep bounty HTML previews next to each issue and load Inter once from design/mocks/shared so every mock stays consistent without Google Fonts.
Drop the old repo-root HTML and font copies now that design/mocks owns them.
Twin fee/stake odds tiles, binding-only tint with a top fade, and aligned “caps rewards” copy so the mock matches how people scan prediction markets.
The whole card is an anchor, so without an explicit color the titles fell back to browser blue on the dark panel.
Market-board mobile list, binding-only top fade, gateway→fees→stake checklist, and clearer review copy.
A glob in the CSS comment closed early and left invalid tokens before the first @font-face rule.
@vercel

vercel Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
explorer-arbitrum-one Ready Ready Preview Oct 16, 2026 12:59am UTC

Request Review

@thebeyondr

Copy link
Copy Markdown
Collaborator Author

For #802 — please review the mocks in design/mocks/mfs-delegation-802/index.html and drop feedback here (copy, layout, day-one card, color rules). Mockups only; no product implementation yet.

@thebeyondr thebeyondr self-assigned this Oct 12, 2026
@thebeyondr thebeyondr linked an issue Oct 12, 2026 that may be closed by this pull request
@thebeyondr
thebeyondr marked this pull request as ready for review October 12, 2026 21:43
Copilot AI balanced review requested due to automatic review settings October 12, 2026 21:43

Copilot AI 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.

🟡 Changes recommended

Broken navigation, out-of-scope reward-history links, mobile overflow, and accessibility issues remain.

13 open findings
What changed in this PR

Adds static MFS delegation UX mocks for issue #802, shared self-hosted Inter fonts, and repository UI guidance.

Changes:

  • Adds delegation list, detail, status, and onboarding mockups.
  • Documents MFS and accessibility conventions.
  • Shares Inter assets across design previews.
File Description
AGENTS.md Adds UI and MFS design rules.
design/​mocks/​mfs-delegation-802/​index.html Adds MFS delegation mockups and rationale.
design/​mocks/​governance-voting/​index.html Adds the governance preview using shared Inter.
design/​mocks/​shared/​inter.css Defines shared Inter font faces.
design/​mocks/​shared/​fonts/​LICENSE Includes the Inter license.
design/​mocks/​shared/​fonts/​inter-latin-400-normal.woff2 Adds Inter 400.
design/​mocks/​shared/​fonts/​inter-latin-500-normal.woff2 Adds Inter 500.
design/​mocks/​shared/​fonts/​inter-latin-600-normal.woff2 Adds Inter 600.
design/​mocks/​shared/​fonts/​inter-latin-700-normal.woff2 Adds Inter 700.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread design/mocks/governance-voting/index.html Outdated
Comment thread design/mocks/governance-voting/index.html
Comment thread design/mocks/governance-voting/index.html
Comment thread design/mocks/governance-voting/index.html
Comment thread design/mocks/mfs-delegation-802/index.html Outdated
Comment thread design/mocks/mfs-delegation-802/index.html
Comment thread design/mocks/mfs-delegation-802/index.html Outdated
Comment thread design/mocks/mfs-delegation-802/index.html Outdated
Comment thread AGENTS.md Outdated
Comment thread design/mocks/mfs-delegation-802/index.html Outdated
Tighten mock accessibility and layout to match AGENTS rules: skip links,
reduced motion, hit targets, zoomable viewport guidance, and help triggers.
@ECWireless

Copy link
Copy Markdown
Collaborator

@thebeyondr some notes:

  • Are tooltips supposed to work already? I'm trying to test them now, but they don't pop up
  • I can see some stakeholders potentially having issue with some columns being swapped out for others. Right now, the explorer has Forecasted Yield, Delegated Stake, and Trailing 90 Day Fees; this design would replace those with State, Fees vs Stake, Reward Cap, and Node Keeps. Only one that might be worth keeping (or incorporating) in my mind is Forecasted Yield. Thoughts @rickstaa ?
  • For the "Orchestrator detail" section, it looks good to me in isolation (as it what and how the data renders looks right), but I'm having trouble mapping this onto how the current orchestrating page looks
  • "Your delegation" looks good to me, but also wondering where the card would fit on the page
  • The "Day one" checklist is cool. Curious @rickstaa 's take on it

Reviewers need hover/focus tooltips on the list help markers, and the
delegating card should speak to an already-bonded stake, not leftover room.
Keep MFS decision columns first, demote yield/stake/fees as quieter
context, and drop ranks and the sort banner.
Show the reward-cap card as the top span of the Orchestrating 2-column
grid with a right-rail amount box, matching widths and notes.

Copilot AI 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.

Comment thread design/mocks/mfs-delegation-802/index.html Outdated
Comment thread design/mocks/governance-voting/index.html
@thebeyondr

Copy link
Copy Markdown
Collaborator Author

The PR is described as MFS-only … Move the governance design to a separate PR

Keeping the governance mock in this PR on purpose: it is the shared-font regression host for design/mocks/shared/, not a separate #802 scope claim. Happy to split later if review noise is still too high.

Copilot AI 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.

🟡 Changes recommended

Mobile delegation currently ends at inactive drawer controls, and several navigation and safe-area issues remain.

1 open finding
8 resolved since last review
Previously missed (2)

In code that hasn't changed since last review

Medium severity Fixed padding ignores safe-area insets

design/​mocks/​governance-voting/​index.html:44

viewport-fit=cover opts into display cutouts, but this fixed padding ignores safe-area insets. In landscape or standalone mode, content can sit under a notch, contrary to the new safe-area rule.

Low severity Opening sentence lacks punctuation

AGENTS.md:1

This opening sentence is missing punctuation between the two clauses.

🧠 Review effort: Balanced

Comment thread design/mocks/mfs-delegation-802/index.html Outdated
Drop violet for stake; use brand green for LPT/shares and brand blue for
fees. Update AGENTS and rationale copy so green means quantity, not good.

Copilot AI 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.

🟡 Changes recommended

Misleading reward copy and accessibility issues remain in the design preview.

4 open findings

🧠 Review effort: Balanced

Comment thread design/mocks/mfs-delegation-802/index.html
Comment thread design/mocks/mfs-delegation-802/index.html
Comment thread design/mocks/mfs-delegation-802/index.html Outdated
Mobile Delegate opens the amount widget as a bottom sheet. Toolbar help
opens below and stays in-frame. Docs step keeps native link semantics.
Balanced examples use equal fee/stake shares so copy matches MFS.
Cliff notes + PDF for 802/803/804; do not counter the litepaper.

Copilot AI 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.

🟡 Changes recommended

Reward-cap aggregation guidance and mobile drawer positioning and focus handling need correction.

3 open findings
4 resolved since last review

🧠 Review effort: Balanced

- derived fee share, stake share, `min()`, gap
- reward cut, fee cut (already present)

Per-round series of those same fields is 803's schema, not a second one. Design 802's window as a sum over the rounds 803 will store. Validator score fields belong to 804, not this ticket.
}
.page-frame.is-drawer-open .pf-rail {
display: block;
position: fixed;
Comment on lines +3741 to +3755
var drawer = frame.querySelector(".pf-drawer");
var toggle = drawer && drawer.querySelector("[data-drawer-focus]");
frame.classList.toggle("is-drawer-open", open);
if (drawer) drawer.classList.toggle("is-open", open);
if (toggle) {
toggle.setAttribute("aria-expanded", open ? "true" : "false");
toggle.setAttribute(
"aria-label",
open ? "Close amount drawer" : "Open amount drawer"
);
}
if (open) {
var input = amountInput(frame);
if (input) input.focus({ preventScroll: true });
}
Match 803 chip borders and ≥24px help targets; convert filter/action segs to radiogroups with keyboard; document icons + APG notes.

Copilot AI 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.

🔵 Needs a closer look

The preview still has unresolved drawer positioning and keyboard-focus behavior.

3 open findings
Previously missed (1)

In code that hasn't changed since last review

Low severity Fix PDF link path to the repository root

design/​mocks/​mfs-delegation-802/​802-mfs-delegation-ux.md:6

The PDF link resolves to design/Livepeer_2.0_Litepaper_September_2026_Draft.pdf, but the file is at the repository root. Add one more parent-directory segment so reviewers can open the protocol reference from this rationale.

🧠 Review effort: Balanced

Copilot AI 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.

🔵 Needs a closer look

The mobile amount drawer still has positioning and keyboard-focus defects.

3 open findings

🧠 Review effort: Balanced

This branch was successfully deployed

1 active deployment
Preview — 204189d4 Deployed Oct 16, 2026 by vercel[bot]
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.

Design new UX for MFS changes

3 participants