Page MenuHomePhabricator

Conpherence - move crumb-based room edit to widget; other cleanup
ClosedPublic

Authored by btrahan on Jun 25 2015, 10:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 11:24 AM
Unknown Object (File)
Thu, Dec 19, 5:20 PM
Unknown Object (File)
Mon, Dec 9, 4:11 PM
Unknown Object (File)
Sun, Dec 8, 4:03 PM
Unknown Object (File)
Thu, Dec 5, 8:32 AM
Unknown Object (File)
Sun, Dec 1, 4:39 AM
Unknown Object (File)
Nov 27 2024, 7:38 AM
Unknown Object (File)
Nov 23 2024, 5:12 AM
Subscribers
Tokens
"Pterodactyl" token, awarded by sshannin.

Details

Summary

We use a non-standard way to invoke the edit dialogue (title, picture, etc) via the crumb. Stop doing that and instead use the widget technology to invoke the dialogue.

This requires making the widget handling code a bit more nuanced as nothing has wanted to pop a dialogue before. I plan to clean this up as I add the action to "Mark as Favorite" to the UI. In particular, I want to stop rendering the un-used DOM and make a workflow-based widget action a property as opposed to something hardcoded. This may be too ambitious depending on how similar these workflows are....

This also updates the ThreadSearchEngine to be a bit more modern. Additionally, go through making some user-facing strings a bit more sensical.

Test Plan

changed settings from conpherence full and durable column successfully.

Diff Detail

Repository
rP Phabricator
Branch
favorite_rooms
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7018
Build 7091: [Placeholder Plan] Wait for 30 Seconds
Build 7090: arc lint + arc unit

Event Timeline

btrahan retitled this revision from to Conpherence - application cleanup.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
btrahan added a subscriber: epriestley.
btrahan retitled this revision from Conpherence - application cleanup to Conpherence - move crumb-based room edit to widget; other cleanup.Jun 25 2015, 10:52 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 25 2015, 10:56 PM
This revision was automatically updated to reflect the committed changes.