Page MenuHomePhabricator

Fix Conpherence layout with setup issues
ClosedPublic

Authored by chad on Nov 27 2013, 11:45 PM.
Tags
None
Referenced Files
F15541784: D7669.diff
Fri, Apr 25, 1:31 PM
F15489644: D7669.id17322.diff
Fri, Apr 11, 11:03 AM
F15487468: D7669.id17319.diff
Thu, Apr 10, 2:19 PM
F15486515: D7669.id.diff
Thu, Apr 10, 7:57 AM
F15482187: D7669.diff
Wed, Apr 9, 3:32 AM
F15458044: D7669.diff
Sun, Mar 30, 8:30 PM
F15412563: D7669.diff
Mar 19 2025, 1:41 PM
Unknown Object (File)
Feb 24 2025, 3:06 AM
Tokens
"Yellow Medal" token, awarded by btrahan.

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPba76526185df: Fix Conpherence layout with setup issues
Summary

Make Conpherence usable when there is a setup issue as well. Fixes T3427

Test Plan

Test an ongoing thread, create a new thread. Test mobile and tablet layouts.

Diff Detail

Branch
conph-setup-div
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

Hmm, I think the thread list doesn't overflow / pick up a scrollbar properly now. I can scroll down the page and have the header vanish:

{F87100}

I guess I didn't check the overloaded cases. And seems like redundant threads continue to be a problem. I wonder if we should just not allow that (hash the participants or something)