Page MenuHomePhabricator
Feed Advanced Search

Feb 6 2017

chad added a revision to T12211: phabricator-home css doesn't get applied to dashboards on home: D17321: More minor CSS tweaks globally.
Feb 6 2017, 6:52 PM · Profile Menu Item, Dashboards, Bug Report

Feb 5 2017

chad added a comment to T12211: phabricator-home css doesn't get applied to dashboards on home.

Or get rid of the darker sidebar. I think it's only needed on projects/profiles

Feb 5 2017, 5:53 PM · Profile Menu Item, Dashboards, Bug Report
epriestley added a comment to T12211: phabricator-home css doesn't get applied to dashboards on home.

(Or whatever more nuanced version of things you want to do. But that's the current fate/rationale of phabricator-home.)

Feb 5 2017, 2:31 PM · Profile Menu Item, Dashboards, Bug Report
epriestley added a comment to T12211: phabricator-home css doesn't get applied to dashboards on home.

.phabricator-home makes the menu full-screen on mobile, so it's currently only applied to /.

Feb 5 2017, 2:30 PM · Profile Menu Item, Dashboards, Bug Report
chad added projects to T12211: phabricator-home css doesn't get applied to dashboards on home: Bug Report, Dashboards, Profile Menu Item.
Feb 5 2017, 7:09 AM · Profile Menu Item, Dashboards, Bug Report
chad created T12208: Collapsable Menu Item.
Feb 5 2017, 5:24 AM · Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Might even contract some illustrations O.o

Feb 5 2017, 12:20 AM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

I plan to go back to Photoshop and re-work dashboards top to bottom... at least, but a lot of time fixing the bad UX everywhere, then ship it as a default application. With Nuance + Facts possibly coming, seems right to get ahead of them.

Feb 5 2017, 12:19 AM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Put this workflow inside "Edit Menu"

Feb 5 2017, 12:12 AM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Two possible alternatives to crumbs I can come up with, although I don't love either:

Feb 5 2017, 12:11 AM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Yeah that is one argument for crumbs

Feb 5 2017, 12:05 AM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

One other thought: it might be nice to provide a way to get to the dashboard from the dashboard-in-a-menu view. Currently, there's no way to "Edit This Dashboard" from Home or Projects: you have to go to Dashboards, hunt it down, and then manage things from there.

Feb 5 2017, 12:04 AM · Dashboards (v2), Profile Menu Item

Feb 4 2017

chad updated the task description for T12202: Upgrade Guide: Home Menu Items.
Feb 4 2017, 3:40 AM · Guides, Installing & Upgrading, Profile Menu Item
rphl added a comment to T12174: Home Menu Errata.

got it! Could reproduce the "restricted error":

Feb 4 2017, 3:15 AM · Dashboards (v2), Profile Menu Item

Feb 3 2017

chad created T12202: Upgrade Guide: Home Menu Items.
Feb 3 2017, 11:48 PM · Guides, Installing & Upgrading, Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17309: Update Profile Menu diviner.
Feb 3 2017, 10:22 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

I'm going to write up Diviner now, then Upgrading guide

Feb 3 2017, 9:30 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Feb 3 2017, 9:29 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Yeah, I don't think I fixed it. I tried this set of steps at master prior to D17308:

Feb 3 2017, 7:01 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

I'm not sure if I actually fixed @rphl's issue or not. D17308 was just aimed at the existing issue under the theory that we'll see more of it after this ships if we aren't already seeing it, but his report definitely might be a new issue.

Feb 3 2017, 6:55 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

I thought it might be that same issue, but sounded like it was a new issue and not a pre-existing one.

Feb 3 2017, 6:30 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Awesome, thanks for hunting that down.

Feb 3 2017, 6:30 PM · Dashboards (v2), Profile Menu Item
epriestley updated the task description for T12174: Home Menu Errata.
Feb 3 2017, 6:27 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17308: When a viewer can't see some of a dashboard's panels, only hide those panels.
Feb 3 2017, 5:58 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17307: Make drag-and-drop to upload files work on any Home dashboard, not just "Magic Home".
Feb 3 2017, 5:22 PM · Dashboards (v2), Profile Menu Item
epriestley merged T12194: Drag-n-drop file upload is broken in HEAD into T12174: Home Menu Errata.
Feb 3 2017, 3:03 AM · Dashboards (v2), Profile Menu Item

Feb 2 2017

chad added a comment to T12174: Home Menu Errata.

Please file a bug report with the steps we can follow to reproduce what your seeing. I still can't follow what I need to do here to see this locally.

Feb 2 2017, 11:19 PM · Dashboards (v2), Profile Menu Item
rphl added a comment to T12174: Home Menu Errata.
  1. If I remove a user from our internal group (Project: contain internal members), I receive the error above.
  2. The internal group is only visible to its members (explains the "restricted project")
  3. Removing the "visible to" restriction, displays Projektname in error message
Feb 2 2017, 11:09 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

It says restricted application and only members of a certain project. Do you have steps I can reproduce the bug locally. I can't reverse guess here offhand how you've configured your instance to get here.

Feb 2 2017, 4:12 PM · Dashboards (v2), Profile Menu Item
rphl added a comment to T12174: Home Menu Errata.

Got today a "Restricted" Report for "Home" from one of our users. Hmm.

Feb 2 2017, 3:52 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

I think D17298 is probably a good change for Favorites either way (so we don't have a "pin" UI there that doesn't do anything) since it's pretty simple.

Feb 2 2017, 3:44 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17298: Remove menu item pinning from Home and Favorites profile menus.
Feb 2 2017, 3:40 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Pinning on Favorites has no effect.

Feb 2 2017, 3:30 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Do we need pinning at all on Home or Favorites? I think it's only for Workboards?

Feb 2 2017, 3:28 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

You want to use top-most displayable even if the user has two personal dashboards and has explicitly clicked the pin icon on the second one to make it their default item?

Feb 2 2017, 3:27 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Basically I had this happen to me, thats why I filed it. Users just know they had a default page they installed and it suddenly went away as default - admins maybe figure out they messed up, but can't un-pin. So all users must then go and pin a default to fix.

Feb 2 2017, 3:27 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

You can change the rules for picking a default item in PhabricatorProfileMenuEngine->loadItems() if you want to use different rules.

Feb 2 2017, 3:26 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

I assumed we were going with top-most displayable, which is simple and admins can't really force you into something else by mistake?

Feb 2 2017, 3:23 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

If you're a user and an admin pins a default, it's confusing and likely not expected.

Feb 2 2017, 3:20 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

There isn't a way to un-set a default, or pick a different global default as a default, although you can effectively do either by creating a copy of whatever global item you want to be default and pinning it, except "magic home", which you can't copy. So there are a couple of messy edge cases there but they don't feel too important to me.

Feb 2 2017, 3:19 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

If Admin "sets default" on Global, and users have not set a default, the page follow admin default. (I had to set my dashboard as default, as seems to be no way of un-setting a default).

Feb 2 2017, 3:16 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Feb 2 2017, 2:57 AM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

I think D17294 + guidance is maybe good enough?

Feb 2 2017, 1:14 AM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Not sure if you pulled today or yesterday, but we've had a dozen diffs on the new menu.

Feb 2 2017, 12:08 AM · Dashboards (v2), Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17294: Add better description to built-in Home.
Feb 2 2017, 12:03 AM · Dashboards (v2), Profile Menu Item

Feb 1 2017

chad updated the task description for T12174: Home Menu Errata.
Feb 1 2017, 11:53 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

@epriestley This part of the migration might be confusing, should we disable the built-in home if they had a Default Dashboard?

Feb 1 2017, 11:48 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

OK, yes, that is the default "Builtin" one we ship with. It wasn't previously visible if you installed a global default. When you migrate to HEAD, it should have ended up with 2 links, one for the built-in home, and one for the global dashboard that was migrated. You can disable the builtin Home under Global Menu Item.

Feb 1 2017, 11:47 PM · Dashboards (v2), Profile Menu Item
rphl added a comment to T12174: Home Menu Errata.

Before "Home Menu"

Feb 1 2017, 11:44 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

You'd need to tell me more about how your install was previously, in case we missed something with migration.

Feb 1 2017, 11:35 PM · Dashboards (v2), Profile Menu Item
rphl added a comment to T12174: Home Menu Errata.

Ok. But I see two different "Home" Dashboards on our install. It is expected?

Feb 1 2017, 11:33 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Theres a lot of flexibility here, so hollar if you get stuck trying to implement something.

Feb 1 2017, 11:31 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

You can now have multiple dashboards, and I think we'll default to the top-most.

Feb 1 2017, 11:30 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

I'll write up a blog post this week and I suspect we'll have a migration guide since it's new to old admins.

Feb 1 2017, 11:28 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

It's no longer a config, nor do you need to "Install Dashboard", Just add your new one to the sidenav and disable the built in.

Feb 1 2017, 11:27 PM · Dashboards (v2), Profile Menu Item
rphl added a comment to T12174: Home Menu Errata.

Is there a way to change the default dashboard used for Home? Couldnt find a config for this.

Feb 1 2017, 11:26 PM · Dashboards (v2), Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17292: Unselect "home" on mobile css.
Feb 1 2017, 10:19 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Feb 1 2017, 7:03 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Feb 1 2017, 6:27 PM · Dashboards (v2), Profile Menu Item
epriestley updated the task description for T12174: Home Menu Errata.
Feb 1 2017, 6:18 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17290: Restore old Home mobile menu behavior, hide crumbs.
Feb 1 2017, 6:18 PM · Dashboards (v2), Profile Menu Item
epriestley updated the task description for T12174: Home Menu Errata.
Feb 1 2017, 6:01 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17289: Allow "Magic Home" to be hidden.
Feb 1 2017, 6:01 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

We have 2 Homes, I'd like 1.

Feb 1 2017, 5:58 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Why do we need to allow it to be hidden?

Feb 1 2017, 5:58 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17288: Show magic "Home" menu item for logged-out users on public installs.
Feb 1 2017, 5:57 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Also need to allow fake 'Home' to be hidden... somehow. I can make a fallback page if needed.

Feb 1 2017, 5:57 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

It looks like it works, it's just a little confusing since "Home" doesn't show up. I'll fix that:

Feb 1 2017, 5:54 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Also, we might be torpedoing open-source installs which don't have a dashboard right now since the magic "Home" doesn't show up if you're logged out. I'll look at that.

Feb 1 2017, 5:53 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

I think that's a whole pile of work, I was just thinking that administrators could disable the global item.

Feb 1 2017, 5:47 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

I looked briefly, but couldn't figure out how to move fake 'Home' into the Personal Menu as a built-in

Feb 1 2017, 5:41 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Leave the default fake 'Home' as is (corporate install, your stuff)

Feb 1 2017, 5:39 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Based on that, the problem to solve is making "contributor" type dashboards easy to find/install.

Feb 1 2017, 5:38 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

So let me walk through what I think you're suggesting.

Feb 1 2017, 5:37 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

I also, generally, can't come up with any uses for links that or other items which are only visible after you log in, outside of dashboards.

Feb 1 2017, 5:33 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

I'd like to wait until we have a good answer on how we'd use this feature before we pursue it. I worry that it adds a significant amount of complexity and may really have zero use cases. In the best case, it is useful to only a tiny number of open source installs.

Feb 1 2017, 5:31 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

As far as Dashboards in general go, I was planning on that next (as a project to clean up) and haven't given a lot of thought about logged in / logged out versions. I have no idea how many installs rely on these, so assumed at least, we weren't removing the feature.

Feb 1 2017, 5:31 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

As a user, being able to hide global links is probably going to come up, and is another way to resolve this.

Feb 1 2017, 5:26 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Oh, or is that non-hypothetical?

Feb 1 2017, 5:23 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

You're asking me a question I'd probably put a few weeks in to answer.

Feb 1 2017, 5:23 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

How would you like the two dashboards to differ on this install?

Feb 1 2017, 5:21 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

The main thing right now is we have two different Home dashboards. If there was a way to resolve that to just one (maybe I just do this in PhabricatorHomeMenuItem?) then we shouldn't need public/all user menu items. At least, I'd wait for more use cases like you suggest.

Feb 1 2017, 5:18 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

Oh, maybe I misunderstood then. How would you like to make the logged-out vs logged-in menus different on this install, or how do you think other open source installs would benefit from this? (This only impacts open-source installs, right?)

Feb 1 2017, 5:15 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

I was referring to menu items. If we can fix up Dashboards enough that (logged in/logged out) is a single dashboard, I think that resolves the immediate use case.

Feb 1 2017, 5:12 PM · Dashboards (v2), Profile Menu Item
epriestley added a comment to T12174: Home Menu Errata.

I'm not really sold on the use case -- which is just letting open source installs have a logged-in dashboard tailored to the user, right?

Feb 1 2017, 5:09 PM · Dashboards (v2), Profile Menu Item
chad added a comment to T12174: Home Menu Errata.

Are we comfortable for Global menus of giving admins "visible (logged in), visible (logged out), visible (all), and hidden" as visibility options? Or just implement Policy and really confuse everything?

Feb 1 2017, 4:59 PM · Dashboards (v2), Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17287: Use a label for default home menu.
Feb 1 2017, 6:17 AM · Dashboards (v2), Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17286: Crisp CSS rules for JX.Tooltip.
Feb 1 2017, 6:04 AM · Dashboards (v2), Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17285: Add some style to label in Favorites Menu.
Feb 1 2017, 3:50 AM · Dashboards (v2), Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17284: Fix application name toolip hover.
Feb 1 2017, 3:36 AM · Dashboards (v2), Profile Menu Item
chad closed T11957: Add MenuEngine to Home as Resolved.

Resolved at HEAD. Follow T12174 for bugs/feature requests/future action items.

Feb 1 2017, 12:12 AM · Profile Menu Item, Dashboards

Jan 31 2017

epriestley updated the task description for T12174: Home Menu Errata.
Jan 31 2017, 9:38 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17281: Improve page titles for menu items that render in-context content.
Jan 31 2017, 9:30 PM · Dashboards (v2), Profile Menu Item
epriestley added a revision to T12174: Home Menu Errata: D17280: Allow menu items to be edited again.
Jan 31 2017, 9:26 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Jan 31 2017, 9:19 PM · Dashboards (v2), Profile Menu Item
chad added a revision to T12174: Home Menu Errata: D17278: Add Label MenuItem.
Jan 31 2017, 9:06 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Jan 31 2017, 8:54 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Jan 31 2017, 8:51 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Jan 31 2017, 8:42 PM · Dashboards (v2), Profile Menu Item
chad updated the task description for T12174: Home Menu Errata.
Jan 31 2017, 8:40 PM · Dashboards (v2), Profile Menu Item