Skip to content

Conversation

@suzyh
Copy link
Collaborator

@suzyh suzyh commented Feb 13, 2017

@shans
Copy link
Owner

shans commented Feb 13, 2017

Can you check that this will fit on the screen before we merge & deploy?

@suzyh
Copy link
Collaborator Author

suzyh commented Feb 13, 2017

Hmm, looks like the version that's displaying up outside your area is different from this config; fewer people listed. So I'm not sure. Who owns the config that's being used?

@shans
Copy link
Owner

shans commented Feb 13, 2017

Probably Doug. I'd prefer us to get back to having our config checked in though (at least until we move away from that altogether). Is it just the listed people that is different, and if so do you mind rolling that into this change too?

@dstoc
Copy link
Collaborator

dstoc commented Feb 13, 2017

The only difference is probably the people. I'm not sure where the current config came from.

And while we're at it, add style folks to the animations dash.
@suzyh
Copy link
Collaborator Author

suzyh commented Feb 13, 2017

Thinking about it some more, I figured it made sense to create a new config. When Style and Animations move to sit together, I think it makes sense to have their dashboards side-by-side, and at that point we can start reducing duplicated info from the two. (I don't think there's room at present for a single dashboard to have both.)

@shans
Copy link
Owner

shans commented Feb 13, 2017 via email

@suzyh
Copy link
Collaborator Author

suzyh commented Feb 13, 2017

I just meant literally having the Animations chromebox + screen next to the Style one. Equipment availability permitting.

@shans
Copy link
Owner

shans commented Feb 13, 2017 via email

@suzyh
Copy link
Collaborator Author

suzyh commented Mar 24, 2017

Judging by the stylimations dashboard today, it looks like this pull request has been made redundant. Feel free to close if so.

It would be nice to make better use of the two screens, e.g. having one screen display fixed cards and the other have the cardview, but from my limited poking at the dashboard code I haven't been able to work out how to control the cardview sufficiently to do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants