Page MenuHomePhabricator
Differential D12602

[web] fix colors in calendar history modal
ClosedPublic
Actions

Authored by ginsu on Thu, Jun 27, 6:47 PM.
Tags
None
Referenced Files
F2201589: D12602.id41870.diff
Sat, Jul 6, 4:22 AM
F2200822: D12602.id41778.diff
Sat, Jul 6, 1:16 AM
F2195310: D12602.id41915.diff
Fri, Jul 5, 10:00 AM
Unknown Object (File)
Thu, Jul 4, 9:58 AM
Unknown Object (File)
Wed, Jul 3, 11:54 AM
Unknown Object (File)
Wed, Jul 3, 11:54 AM
Unknown Object (File)
Wed, Jul 3, 11:54 AM
Unknown Object (File)
Wed, Jul 3, 11:53 AM
View All 13 Files
Subscribers
ashoat
tomek

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
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ginsu created this revision.Thu, Jun 27, 6:47 PM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Jun 27, 6:47 PM
ginsu edited the summary of this revision. (Show Details)Thu, Jun 27, 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.Thu, Jun 27, 7:05 PM
ginsu requested review of this revision.Thu, Jun 27, 7:05 PM
ashoat accepted this revision.Mon, Jul 1, 1:23 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.Mon, Jul 1, 1:23 PM
ginsu updated this revision to Diff 41870.Mon, Jul 1, 10:01 PM

address comments

ginsu edited the test plan for this revision. (Show Details)Mon, Jul 1, 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.Mon, Jul 1, 10:18 PM
Closed by commit rCOMMfc09fb873d01: [web] fix colors in calendar history modal (authored by ginsu). · Explain WhyTue, Jul 2, 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

PathSize
web/
modals/
history/
15 lines

Diff 41915

View Options

web/modals/history/history.css

Loading...