[web] Address remaining code review comments from D3341
ClosedPublic
Actions

Authored by tomek on Mar 9 2022, 5:55 PM.

Details

Reviewers

• benschac
atul
• jacek

Commits

rCOMM2f88cd59f456: [web] Address remaining code review comments from D3341

Summary

Rename activeThreadCurrentlyUnread and delete unused variable
Depends on D3341
Depends on D3387

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Mar 9 2022, 5:55 PM

Herald added subscribers: • karol, • adrian, ashoat. · View Herald TranscriptMar 9 2022, 5:55 PM

tomek mentioned this in D3341: [web] Make NavigationPanel reusable.Mar 9 2022, 5:55 PM

Harbormaster completed remote builds in B7261: Diff 10232.Mar 9 2022, 6:00 PM

tomek requested review of this revision.Mar 9 2022, 6:00 PM

atul accepted this revision.Mar 9 2022, 6:17 PM

This revision is now accepted and ready to land.Mar 9 2022, 6:17 PM

• benschac accepted this revision.Mar 9 2022, 6:30 PM

tomek added a child revision: D3342: [web] Introduce settings switcher.Mar 11 2022, 9:43 AM

Closed by commit rCOMM2f88cd59f456: [web] Address remaining code review comments from D3341. · Explain WhyMar 11 2022, 11:45 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM2f88cd59f456: [web] Address remaining code review comments from D3341.

Revision Contents
Changeset List

Path

Size

web/

sidebar/

app-switcher.react.js

12 lines

Diff 10292

View Options

web/sidebar/app-switcher.react.js

[web] Address remaining code review comments from D3341ClosedPublicActions