Page MenuHomePhabricator
Differential D13274

Update threadIsPendingSidebar to handle thick sidebars
ClosedPublic
Actions

Authored by marcin on Tue, Sep 10, 7:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 27, 10:51 PM
Unknown Object (File)
Wed, Sep 25, 9:34 AM
Unknown Object (File)
Sun, Sep 22, 12:31 AM
Unknown Object (File)
Sat, Sep 21, 7:18 PM
Unknown Object (File)
Sat, Sep 21, 6:53 PM
Unknown Object (File)
Sat, Sep 21, 4:03 PM
Unknown Object (File)
Sat, Sep 21, 3:12 PM
Unknown Object (File)
Sat, Sep 21, 3:44 AM
View All 37 Files
Subscribers
ashoat

Details

Reviewers
inka
tomek
Commits
rCOMM085726ba64cd: Update threadIsPendingSidebar to handle thick sidebars
Summary

This differential updates threadIsPendingSidebar to handle thick sidebars as well

Test Plan

Test was added to thread-utils.test.js

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

marcin created this revision.Tue, Sep 10, 7:07 AM
Herald added a subscriber: ashoat. · View Herald TranscriptTue, Sep 10, 7:07 AM
Harbormaster completed remote builds in B31508: Diff 44009.Tue, Sep 10, 7:28 AM
marcin requested review of this revision.Tue, Sep 10, 7:28 AM
inka accepted this revision.Wed, Sep 11, 2:07 AM

Thank you!

This revision is now accepted and ready to land.Wed, Sep 11, 2:07 AM
tomek accepted this revision.Wed, Sep 11, 2:08 AM
tomek added inline comments.
lib/shared/thread-utils.js
377–378 ↗(On Diff #44009)

Can we extract sidebar and dm_sidebar somewhere as constants?

marcin updated this revision to Diff 44036.Wed, Sep 11, 4:42 AM

Extract sidebar and dm_sidebar to constants

Harbormaster completed remote builds in B31530: Diff 44036.Wed, Sep 11, 4:59 AM
marcin updated this revision to Diff 44040.Wed, Sep 11, 7:02 AM

rebase before landing

This revision was landed with ongoing or failed builds.Wed, Sep 11, 7:04 AM
Closed by commit rCOMM085726ba64cd: Update threadIsPendingSidebar to handle thick sidebars (authored by marcin). · Explain Why
This revision was automatically updated to reflect the committed changes.
marcin added a commit: rCOMM085726ba64cd: Update threadIsPendingSidebar to handle thick sidebars.
Harbormaster completed remote builds in B31533: Diff 44040.Wed, Sep 11, 7:30 AM

Revision Contents
Changeset List

PathSize
lib/
shared/
22 lines
26 lines
utils/
6 lines

Diff 44041

View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/shared/thread-utils.test.js

Loading...
View Options

lib/utils/validation-utils.js

Loading...