-
Notifications
You must be signed in to change notification settings - Fork 383
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
Frontend of iOS/iPadOS as platforms/labels #20157
Conversation
ca58f31
to
63279f6
Compare
frontend/components/PlatformCompatibility/PlatformCompatibility.tsx
Outdated
Show resolved
Hide resolved
frontend/pages/DashboardPage/cards/HostsSummary/HostsSummary.tsx
Outdated
Show resolved
Hide resolved
frontend/pages/ManageControlsPage/OSSettings/cards/CustomSettings/CustomSettings.tsx
Outdated
Show resolved
Hide resolved
frontend/pages/ManageControlsPage/OSUpdates/components/EmptyTargetForm/EmptyTargetForm.tsx
Outdated
Show resolved
Hide resolved
frontend/pages/ManageControlsPage/OSUpdates/components/TargetSection/TargetSection.tsx
Outdated
Show resolved
Hide resolved
frontend/pages/ManageControlsPage/OSUpdates/components/PlatformTabs/PlatformTabs.tsx
Outdated
Show resolved
Hide resolved
frontend/pages/ManageControlsPage/OSUpdates/components/PlatformTabs/PlatformTabs.tsx
Outdated
Show resolved
Hide resolved
...eControlsPage/OSSettings/cards/CustomSettings/components/ProfileListItem/ProfileListItem.tsx
Outdated
Show resolved
Hide resolved
frontend/pages/admin/TeamManagementPage/TeamDetailsWrapper/TeamDetailsWrapper.tsx
Show resolved
Hide resolved
393681c
to
0f7907a
Compare
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.
Looks good, one small typing improvement which we can make in a future iteration.
@@ -45,10 +45,16 @@ const HostsSummary = ({ | |||
opacity = isLoadingHostsSummary ? { opacity: 0.4 } : { opacity: 1 }; | |||
} | |||
|
|||
const getBuiltinLabelId = useCallback( |
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.
🙌
@@ -22,12 +22,12 @@ const DISPLAY_ORDER = [ | |||
"Windows", | |||
"Linux", | |||
"ChromeOS", | |||
] as SupportedDisplayPlatform[]; | |||
] as QueryableDisplayPlatform[]; |
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.
For speed this can be on another iteration but can we still improve the typing on line 14?
Issue
Frontend of story #19962
Description
Lots of page changes
Screenshots and screen recordings
Early recording with ~1/2 the changes in
https://www.loom.com/share/e9d8c4c934634f4abfe04369ca35dd36?sid=4c1527ad-a923-4183-b1fe-df34890be6c6
Updates to the OS Updates page
Everything working on @lucasmrod 's integration test 7/3/24 1:15 EST
https://www.loom.com/share/e32739f113fe4a0eb275e307578ff367?sid=4f2e601e-de7a-4614-88d1-dd5495a4bed7
Checklist for submitter
If some of the following don't apply, delete the relevant line.
On backend ticket