Hide next/prev block buttons on mobile lists and search results #1219
Conversation
Deployed to Cloudflare Pages
|
| @@ -89,8 +89,12 @@ export const BlockDetailView: FC<{ | |||
| <dd> | |||
| <BlockLink scope={block} height={block.round} /> | |||
| <CopyToClipboard value={block.round.toString()} /> | |||
| <PrevBlockButton scope={block} currentRound={block.round} /> | |||
| <NextBlockButton scope={block} currentRound={block.round} /> | |||
| {!standalone && ( | |||
There was a problem hiding this comment.
Issue: https://5603e983.oasis-explorer.pages.dev/search?q=2%2C265%2C700
still appears on search results too
There was a problem hiding this comment.
(sidenote, i'm confused what standalone means.)
There was a problem hiding this comment.
I would expect standalone to mean single block view, not the opposite https://5603e983.oasis-explorer.pages.dev/mainnet/sapphire/block/2265700
There was a problem hiding this comment.
hard to say what it means, but I think naming does not matter now. Let's fix buttons and imo let's refactor vertical tables to rely only on css. This way we will get rid of that prop (unless it is used in more contexts that I am not aware of)
There was a problem hiding this comment.
added enableBlockNavigation prop
buberdds
force-pushed
the
mz/fixBlockButtons
branch
from
5a79d75 to
234bff3
Compare
buberdds
changed the title
buberdds
force-pushed
the
mz/fixBlockButtons
branch
from
234bff3 to
2c031f2
Compare
Closes #1221