-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Working triptych app skeleton with viewport shifter, request logging, packaging instructions #1833
feat: Working triptych app skeleton with viewport shifter, request logging, packaging instructions #1833
Changes from all commits
5bc8975
5154c37
c2938b7
89b2d5b
83dbcfb
7182494
5ece363
5f3f985
fa3f629
90f0320
7f7744f
10653e6
6de37c2
a28d258
88e5511
7db3eae
b1064f1
99c181a
1605140
284a5ca
4189456
a0171c0
4ea45fa
f775909
e669c59
e157570
f995910
e9c6a03
72059da
1005d8b
9e76838
f266e7c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,24 +1,8 @@ | ||
#debug { | ||
position: absolute; | ||
bottom: 0; | ||
right: 0; | ||
width: 100%; | ||
background: rgba(0, 0, 0, 0.5); | ||
color: #fff; | ||
display: flex; | ||
flex-wrap: wrap; | ||
flex-direction: column; | ||
height: 100%; | ||
font-size: 1.5em; | ||
z-index: 10000; | ||
#debug.dup { | ||
width: 1920px; | ||
height: 1080px; | ||
} | ||
|
||
#debug p { | ||
position: relative; | ||
z-index: 9999; | ||
font-size: 1.5em; | ||
bottom: unset; | ||
left: unset; | ||
letter-spacing: normal; | ||
font-family: Arial; | ||
#debug .line { | ||
width: 640px; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
#debug { | ||
position: absolute; | ||
top: 0; | ||
left: 0; | ||
z-index: 10000; | ||
overflow-y: hidden; | ||
overflow-x: hidden; | ||
|
||
background: rgba(0, 0, 0, 0.5); | ||
|
||
display: flex; | ||
flex-flow: column wrap; | ||
|
||
color: #fff; | ||
font-size: 1.5em; | ||
font-family: monospace; | ||
} | ||
|
||
#debug .line { | ||
box-sizing: border-box; | ||
padding-left: 8px; | ||
overflow-wrap: break-word; | ||
} |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
#debug.triptych { | ||
width: 1080px; | ||
height: 1920px; | ||
} | ||
|
||
#debug .line { | ||
width: 540px; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
.no-data-middle { | ||
position: absolute; | ||
top: 0px; | ||
left: 1080px; | ||
} | ||
|
||
.no-data-right { | ||
position: absolute; | ||
top: 0px; | ||
left: 2160px; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
.triptych-screen-viewport { | ||
width: 1080px; | ||
height: 1920px; | ||
overflow: hidden; | ||
position: relative; | ||
margin-left: auto; | ||
margin-right: auto; | ||
|
||
.triptych-shifter { | ||
width: 3240px; | ||
height: 1920px; | ||
overflow: hidden; | ||
position: absolute; | ||
} | ||
|
||
.triptych-shifter--left { | ||
left: 0; | ||
} | ||
|
||
.triptych-shifter--middle { | ||
left: -1080px; | ||
} | ||
|
||
.triptych-shifter--right { | ||
right: 0px; | ||
} | ||
} | ||
|
||
.triptych-screen-viewport--all { | ||
width: 3240px; | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -17,7 +17,7 @@ import { | |
MultiRotationPage, | ||
SimulationPage, | ||
} from "Components/dup/dup_screen_page"; | ||
import { isDup } from "Util/util"; | ||
import { isDup } from "Util/outfront"; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I unresolved a previous comment thread to ask about this in better detail. (Just noting here so you don't miss it.) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, if I hadn't updated this and other references to The real reason it's updated though, is simply that it's easier to do so. I used the vscode refactor function (F2) to rename the function as well as automatically update all references to it. It would have been harder to go back afterward and undo the renames in the old files. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Love it! That's a great reason. |
||
|
||
const App = (): JSX.Element => { | ||
if (isDup()) { | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll try to get this reviewed / merged before the no-data branch. It's possible these may / may not be needed depending on the implementation of that, but we can leave that for the no-data pr to resolve.