Dynamically calculate scrollbar on /spec #250

Closed
opened 2019-02-07 18:33:40 +01:00 by kauffj · 3 comments
kauffj commented 2019-02-07 18:33:40 +01:00 (Migrated from github.com)

Currently, lbry.tech/spec is showing a double scrollbar: one for the iframe, and one for the site itself.

Currently, lbry.tech/spec is showing a double scrollbar: one for the iframe, and one for the site itself.
NetOpWibby commented 2019-02-07 18:51:46 +01:00 (Migrated from github.com)

Closed via 241926c649

Closed via https://github.com/lbryio/lbry.tech/commit/241926c649435223919e84ab720df4bdd53742fb
lyoshenka commented 2019-02-07 22:25:25 +01:00 (Migrated from github.com)

@NetOperatorWibby i think the real way to fix this is to fix my hacky css in app/views/spec.js. The iframe size should not be hardcoded as <div style="width: 100%; height: calc(100vh - 4rem)">. It should be styled using sass, with the height calculated based on the size of the nav bar.

@NetOperatorWibby i think the real way to fix this is to fix my hacky css in app/views/spec.js. The iframe size should not be hardcoded as `<div style="width: 100%; height: calc(100vh - 4rem)">`. It should be styled using sass, with the height calculated based on the size of the nav bar.
NetOpWibby commented 2019-02-07 22:44:55 +01:00 (Migrated from github.com)

Okay. The issue is fixed though.

EDIT: I'll fix the styling soon.

Okay. The issue _is_ fixed though. EDIT: I'll fix the styling soon.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry.tech#250
No description provided.