Skip to content

Tweaks around ROFL apps search results#2358

Merged
buberdds merged 1 commit into
masterfrom
mz/roflSearchResults
Nov 21, 2025
Merged

Tweaks around ROFL apps search results#2358
buberdds merged 1 commit into
masterfrom
mz/roflSearchResults

Conversation

@buberdds

@buberdds buberdds commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

Closes #2357

  • we need to keep "View App" button to keep search consistent. I am not sure removing it everywhere is expected.
  • keeping "ParaTime" line below app name. Scoped search return all networks for a given layer + global search return all networks for all layers. It is easy for users to get lost without this row.
  • name copy to clipboard copies app id, right?

global search
https://pr-2358.oasis-explorer.pages.dev/search?q=demo-rofl

testnet scoped search (mainnet still avail at the bottom/collapsible section)
https://pr-2358.oasis-explorer.pages.dev/testnet/sapphire/search?q=demo-rofl

@github-actions

github-actions Bot commented Nov 21, 2025

Copy link
Copy Markdown

Deployed to Cloudflare Pages

Latest commit: c21d6b39e87d5194097061db6d6dcb1d2c021681
Status:✅ Deploy successful!
Preview URL: https://a42ae956.oasis-explorer.pages.dev
Alias: https://pr-2358.oasis-explorer.pages.dev

network={app.network}
withSourceIndicator={false}
/>
<CopyToClipboard value={app.id} />

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would copy the app name, because you may search for a substring and the app name may be way longer and you want to copy its name, e.g. to avoid typos.

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.

I think this will make #2332 more confusing

@buberdds buberdds force-pushed the mz/roflSearchResults branch from 10fd8f4 to 3f6a46f Compare November 21, 2025 11:45
@buberdds buberdds force-pushed the mz/roflSearchResults branch from 3f6a46f to c21d6b3 Compare November 21, 2025 11:47
@buberdds buberdds merged commit bb84916 into master Nov 21, 2025
10 checks passed
@buberdds buberdds deleted the mz/roflSearchResults branch November 21, 2025 11:51
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.

[UX]: A few nits in the ROFL search results

4 participants