Skip to content

Conversation

@jjscheel
Copy link
Contributor

@jjscheel jjscheel commented Sep 9, 2025

Added Community, Source Repo, and version history.

Added Community, Source Repo, and version history.

Signed-off-by: Jeff Scheel <[email protected]>
@BlackstoneEngineering
Copy link
Collaborator

Screenshot 2025-09-09 at 9 13 06 AM
  • Technically nothing wrong
  • Style-wise, should we move this information below the PDF or above it?

@jjscheel
Copy link
Contributor Author

jjscheel commented Sep 9, 2025

Technically nothing wrong

Good. Thanks.

Style-wise, should we move this information below the PDF or above it?

I feel that this page should not be viewed on this page now that it's become a details page. SO, that begs the question of whether NEED to keep the embedded viewing on the PDF. Was there an explicit reason (such as analytics) we embedded the viewing of the PDF instead of simply linking and letting the browser render?

@BlackstoneEngineering BlackstoneEngineering changed the title Prototype missing details for SBI Jeff's Changes prior to Launch Sep 18, 2025
@BlackstoneEngineering
Copy link
Collaborator

@jjscheel - please pull locally and build to preview, if you're happy then merge it, looks like build checks passed.

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