Differential D12602

[web] fix colors in calendar history modal
ClosedPublic
Actions

Authored by ginsu on Jun 27 2024, 6:47 PM.

Tags

None

Referenced Files

	F3582089: D12602.id41915.diff
	Sun, Dec 29, 10:43 AM

	F3574502: D12602.diff
	Sat, Dec 28, 5:36 PM

	F3573083: D12602.diff
	Sat, Dec 28, 1:59 PM

	Unknown Object (File)
	Fri, Dec 27, 10:50 PM

	Unknown Object (File)
	Fri, Dec 27, 10:50 PM

	Unknown Object (File)
	Fri, Dec 27, 10:50 PM

	Unknown Object (File)
	Fri, Dec 27, 10:50 PM

	Unknown Object (File)
	Mon, Dec 16, 6:50 PM

View All 82 Files

Subscribers

Details

Reviewers

inka
ashoat

Commits

rCOMMfc09fb873d01: [web] fix colors in calendar history modal

Summary

This diff fixes the colors in the calendar history modal

Linear task: https://linear.app/comm/issue/ENG-8355/convert-calendary-history-modal-on-web-to-dark-mode

Test Plan

Please see the screenshots below

Before:

Screenshot 2024-06-27 at 9.09.43 PM.png (2×3 px, 797 KB)

After:

Screenshot 2024-07-02 at 1.00.31 AM.png (2×3 px, 775 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ginsu created this revision.Jun 27 2024, 6:47 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJun 27 2024, 6:47 PM

ginsu edited the summary of this revision. (Show Details)Jun 27 2024, 6:50 PM

ginsu edited the test plan for this revision. (Show Details)

ginsu added a reviewer: inka.

ginsu attached a referenced file: F2131600: Screenshot 2024-06-27 at 9.49.24 PM.png. (Show Details)

ginsu attached a referenced file: F2131597: Screenshot 2024-06-27 at 9.09.43 PM.png. (Show Details)

Harbormaster completed remote builds in B29979: Diff 41778.Jun 27 2024, 7:05 PM

ginsu requested review of this revision.Jun 27 2024, 7:05 PM

Huge improvement! In the old design, the background for "June 1" was different than the background for the rest. Do you think we can preserve that here?

This revision is now accepted and ready to land.Jul 1 2024, 1:23 PM

address comments

ginsu edited the test plan for this revision. (Show Details)Jul 1 2024, 10:05 PM

ginsu attached a referenced file: F2163971: Screenshot 2024-07-02 at 1.00.31 AM.png. (Show Details)

Harbormaster completed remote builds in B30040: Diff 41870.Jul 1 2024, 10:18 PM

Closed by commit rCOMMfc09fb873d01: [web] fix colors in calendar history modal (authored by ginsu). · Explain WhyJul 2 2024, 1:46 PM

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMMfc09fb873d01: [web] fix colors in calendar history modal.

Revision Contents
Changeset List

Path

Size

web/

modals/

history/

15 lines

Diff 41915

web/modals/history/history.css

Loading...