Page MenuHomePhabricator
Differential D8338

[web] Use colors from themes.css in dropdown
ClosedPublic
Actions

Authored by rohan on Jun 27 2023, 11:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 1, 12:56 AM
Unknown Object (File)
Fri, Mar 28, 5:20 AM
Unknown Object (File)
Thu, Mar 27, 3:33 AM
Unknown Object (File)
Mon, Mar 3, 8:20 PM
Unknown Object (File)
Feb 18 2025, 5:43 AM
Unknown Object (File)
Feb 15 2025, 3:57 PM
Unknown Object (File)
Feb 14 2025, 11:58 PM
Unknown Object (File)
Feb 14 2025, 7:18 PM
View All Files
Subscribers
• ted
tomek

Details

Reviewers
atul
ashoat
Commits
rCOMM2bb6d801c1fd: [web] Use colors from themes.css in dropdown
Summary

Some feedback in D8136 was to use colors from themes.css instead of inlining them in SWMansionIcon.

Depends on D8337

Test Plan

Confirmed that both enabled and disabled dropdowns still look the same

Diff Detail

Repository
rCOMM Comm
Branch
change_roles-quickfix
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

rohan created this revision.Jun 27 2023, 11:26 AM
Herald added subscribers: • ted, tomek. · View Herald TranscriptJun 27 2023, 11:26 AM
Harbormaster completed remote builds in B20553: Diff 28187.Jun 27 2023, 11:44 AM
rohan requested review of this revision.Jun 27 2023, 11:44 AM
rohan mentioned this in D8136: [web] Implement a custom dropdown menu component.Jun 27 2023, 12:12 PM
atul accepted this revision.Jun 28 2023, 10:26 AM

Let's make sure to fix typo before landing

web/components/dropdown.react.js
39 ↗(On Diff #28187)

Do we need !!disabled, or can we just do disabled?

web/theme.css
235 ↗(On Diff #28187)

Minor typo, should be dropdown instead of dropddown

This revision is now accepted and ready to land.Jun 28 2023, 10:26 AM
atul added inline comments.Jun 28 2023, 10:28 AM
web/components/dropdown.react.js
39 ↗(On Diff #28187)

I guess we do !!disabled above in the other calls to classNames, so it's probably best to maintain consistency.

rohan added inline comments.Jun 28 2023, 10:34 AM
web/theme.css
235 ↗(On Diff #28187)

Oh yeah thanks for catching that

rohan updated this revision to Diff 28215.Jun 28 2023, 10:35 AM

Typo

This revision was landed with ongoing or failed builds.Jun 28 2023, 10:38 AM
Closed by commit rCOMM2bb6d801c1fd: [web] Use colors from themes.css in dropdown (authored by rohan). · Explain Why
This revision was automatically updated to reflect the committed changes.
rohan added a commit: rCOMM2bb6d801c1fd: [web] Use colors from themes.css in dropdown.
Harbormaster completed remote builds in B20580: Diff 28215.Jun 28 2023, 10:52 AM

Revision Contents
Changeset List

PathSize
web/
components/
1 line
8 lines
1 line

Diff 28215

View Options

web/components/dropdown.css

Show First 20 Lines • Show All 98 Lines • ▼ Show 20 Lines
export type FFmpegStatistics = { export type FFmpegStatistics = {
// seconds of video being processed per second // seconds of video being processed per second
+speed: number, +speed: number,
// total milliseconds of video processed so far // total milliseconds of video processed so far
+time: number, +time: number,
// total result file size in bytes so far // total result file size in bytes so far
+size: number, +size: number,
+videoQuality: number,
+videoFrameNumber: number,
+videoFps: number,
+bitrate: number,
}; };
export type TranscodeVideoMediaMissionStep = { export type TranscodeVideoMediaMissionStep = {
+step: 'video_ffmpeg_transcode', +step: 'video_ffmpeg_transcode',
+success: boolean, +success: boolean,
+exceptionMessage: ?string, +exceptionMessage: ?string,
+time: number, // ms +time: number, // ms
+returnCode: ?number,
+newPath: ?string, +newPath: ?string,
+stats: ?FFmpegStatistics, +stats: ?FFmpegStatistics,
}; };
export type VideoGenerateThumbnailMediaMissionStep = { export type VideoGenerateThumbnailMediaMissionStep = {
+step: 'video_generate_thumbnail', +step: 'video_generate_thumbnail',
+success: boolean, +success: boolean,
+time: number, // ms +time: number, // ms
▲ Show 20 Lines • Show All 698 Lines • Show Last 20 Lines
View Options

web/components/dropdown.react.js

Show First 20 Lines • Show All 98 Lines • ▼ Show 20 Lines
export type FFmpegStatistics = { export type FFmpegStatistics = {
// seconds of video being processed per second // seconds of video being processed per second
+speed: number, +speed: number,
// total milliseconds of video processed so far // total milliseconds of video processed so far
+time: number, +time: number,
// total result file size in bytes so far // total result file size in bytes so far
+size: number, +size: number,
+videoQuality: number,
+videoFrameNumber: number,
+videoFps: number,
+bitrate: number,
}; };
export type TranscodeVideoMediaMissionStep = { export type TranscodeVideoMediaMissionStep = {
+step: 'video_ffmpeg_transcode', +step: 'video_ffmpeg_transcode',
+success: boolean, +success: boolean,
+exceptionMessage: ?string, +exceptionMessage: ?string,
+time: number, // ms +time: number, // ms
+returnCode: ?number,
+newPath: ?string, +newPath: ?string,
+stats: ?FFmpegStatistics, +stats: ?FFmpegStatistics,
}; };
export type VideoGenerateThumbnailMediaMissionStep = { export type VideoGenerateThumbnailMediaMissionStep = {
+step: 'video_generate_thumbnail', +step: 'video_generate_thumbnail',
+success: boolean, +success: boolean,
+time: number, // ms +time: number, // ms
▲ Show 20 Lines • Show All 698 Lines • Show Last 20 Lines
View Options

web/theme.css

Show First 20 Lines • Show All 98 Lines • ▼ Show 20 Lines
export type FFmpegStatistics = { export type FFmpegStatistics = {
// seconds of video being processed per second // seconds of video being processed per second
+speed: number, +speed: number,
// total milliseconds of video processed so far // total milliseconds of video processed so far
+time: number, +time: number,
// total result file size in bytes so far // total result file size in bytes so far
+size: number, +size: number,
+videoQuality: number,
+videoFrameNumber: number,
+videoFps: number,
+bitrate: number,
}; };
export type TranscodeVideoMediaMissionStep = { export type TranscodeVideoMediaMissionStep = {
+step: 'video_ffmpeg_transcode', +step: 'video_ffmpeg_transcode',
+success: boolean, +success: boolean,
+exceptionMessage: ?string, +exceptionMessage: ?string,
+time: number, // ms +time: number, // ms
+returnCode: ?number,
+newPath: ?string, +newPath: ?string,
+stats: ?FFmpegStatistics, +stats: ?FFmpegStatistics,
}; };
export type VideoGenerateThumbnailMediaMissionStep = { export type VideoGenerateThumbnailMediaMissionStep = {
+step: 'video_generate_thumbnail', +step: 'video_generate_thumbnail',
+success: boolean, +success: boolean,
+time: number, // ms +time: number, // ms
▲ Show 20 Lines • Show All 698 Lines • Show Last 20 Lines