Page MenuHomePhabricator

Make it less confusing to create root-level Phriction doc
ClosedPublic

Authored by amckinley on Dec 10 2018, 9:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:45 AM
Unknown Object (File)
Tue, Apr 23, 6:31 PM
Unknown Object (File)
Thu, Apr 4, 2:16 AM
Unknown Object (File)
Mar 19 2024, 10:58 PM
Unknown Object (File)
Mar 19 2024, 10:58 PM
Unknown Object (File)
Mar 19 2024, 10:58 PM
Unknown Object (File)
Mar 19 2024, 10:58 PM
Unknown Object (File)
Mar 19 2024, 10:58 PM
Subscribers

Details

Summary

Without an existing root document, Phriction shows a nice little "fake" document as the landing page, which has its own nice "Edit this document" button. When showing that page, don't also render the standard "New Document" breadcrumb in the top right. That button always prompts first for a slug name, which is silly when the root document doesn't exist (because the slug name is required to be '').

Test Plan

Loaded Phriction with and without a root document.

Diff Detail

Repository
rP Phabricator
Branch
phriction-root (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21282
Build 28955: Run Core Tests
Build 28954: arc lint + arc unit

Event Timeline

Maybe consider setIsWelcomeDocument() or setHideNewDocumentButton(), on first read it was a little surprising that the effect of showWelcomeDocument isn't to cause a welcome document to be shown.

This revision is now accepted and ready to land.Dec 10 2018, 9:49 PM
This revision was automatically updated to reflect the committed changes.