Small design tweaks, 10/25

@Prtksxna, thanks for adding the screenshots. I'm going through this one more time so I can get it estimated, and caught a few things. Please make the following changes.

Edit summary screen

• Please add a new design for the Event Summary page to show it during the "interim" period, where we've changed the navigation but haven't yet changed the data and overall page design. I left a place for it in the Description, above.

Edit list screen

• Need an "Update totals" button on this page, I now see. (I'm not sure why it was not there before, but with the Download menu it is definitely necessary).
• Also please add the "last updated..." anotation to the screen.

All screens, naming

• Update totals Is there a reason why we changed "Update data" to "Update totals"? They seem about the same to me, so I'm OK with it. But on the other hand we shouldn't change things if we don't have a good reason, since this name is probably used in Help docs etc. . If that wasn't intentional, then maybe change it back. Ping me with your decision, since the text of the Description now matches your mocks.
• Settings I did say we could drop the "Event" from the settings button. But I think I may have forgotten that on the Program pages, there is currently an "edit program" link. So we're heading down a road that will bring us to a place where two buttons with the same name will take you to two different pages/toolsets. What is your thought about that?

In T206495#4695416, @jmatazzoni wrote:

Edit list screen

• Also please add the "last updated..." anotation to the screen.

There is a last updated annotation in the text below. I'd like to move it up (closer to the buttons, as it is on the summary page), but we had decided we don't want to touch the content of that page at all so we let it stay there. Happy to move it up and remove it below if you're fine with that.

All screens, naming

• Update totals Is there a reason why we changed "Update data" to "Update totals"? They seem about the same to me, so I'm OK with it. But on the other hand we shouldn't change things if we don't have a good reason, since this name is probably used in Help docs etc. . If that wasn't intentional, then maybe change it back. Ping me with your decision, since the text of the Description now matches your mocks.

Changed it back to Update data, I don't know why we changed it. Maybe on oversight?

• Settings I did say we could drop the "Event" from the settings button. But I think I may have forgotten that on the Program pages, there is currently an "edit program" link. So we're heading down a road that will bring us to a place where two buttons with the same name will take you to two different pages/toolsets. What is your thought about that?

If I remember correctly, we did discussed the following: different aspects of a UI has different settings, and users already have an understanding that the settings (buttons) are for the object that they're looking at. For example, a settings button on a Facebook album, and a settings button on a profile page does different things.

In T206495#4701068, @Prtksxna wrote:

In T206495#4695416, @jmatazzoni wrote:

Edit list screen

• Also please add the "last updated..." anotation to the screen.

There is a last updated annotation in the text below. I'd like to move it up (closer to the buttons, as it is on the summary page), but we had decided we don't want to touch the content of that page at all so we let it stay there. Happy to move it up and remove it below if you're fine with that.

yes, please move it up to be with the buttons, as on other pages. Thanks. Moving this for estimation.

@Prtksxna pls add design specs for changed naming, etc.

PR: https://github.com/wikimedia/eventmetrics/pull/145

I took the liberty of applying the same styling changes to the other pages, just to keep them all consistent. I've got it deployed at https://eventmetrics-dev.wmflabs.org. @Prtksxna Let me know what you think! However note that the "Edit List" page is currently broken until https://github.com/wikimedia/eventmetrics/pull/143 is merged.

Some things feel a little off, like using the large bolden font for "Create an event". I think it's okay?

Also, I went by the mock up shown above that has the top-right buttons ordered like so:

while the mockup at T206576 has:

Personally I prefer the latter, where "Download options" is a normal button in the middle, and the "Update totals" is the primary blue-coloured button on the right (since that button makes changes to the data).

In T206495#4760904, @MusikAnimal wrote:

I took the liberty of applying the same styling changes to the other pages, just to keep them all consistent. I've got it deployed at https://eventmetrics-dev.wmflabs.org. @Prtksxna Let me know what you think! However note that the "Edit List" page is currently broken until https://github.com/wikimedia/eventmetrics/pull/143 is merged.

Whoa! Thank you for making these changes 😊

Some things feel a little off, like using the large bolden font for "Create an event". I think it's okay?

You're right, that shouldn't be bold — https://prtksxna.github.io/wmf-prototype-gm/new-event.html

In T206495#4760943, @MusikAnimal wrote:

while the mockup at T206576 has:

This seems like an outdated version. Where are you seeing this (I should fix it)? The the dev link that you shared has the correct intended ordering.

In T206495#4767043, @Prtksxna wrote:

Some things feel a little off, like using the large bolden font for "Create an event". I think it's okay?

You're right, that shouldn't be bold — https://prtksxna.github.io/wmf-prototype-gm/new-event.html

Okee doke, this is now fixed and can be viewed on https://eventmetrics-dev.wmflabs.org

I completely forgot about https://prtksxna.github.io/wmf-prototype-gm/. It is very helpful!

The other thing I did (before revisiting your prototype) was make program pages look like the event page, with the page title "Event list", e.g.:

And similarly for "Program settings":

Not sure if we really want that or not.

while the mockup at T206576 has:
F26448689

This seems like an outdated version. Where are you seeing this (I should fix it)? The the dev link that you shared has the correct intended ordering.

Hmm, obviously not at T206576. I don't know where I saw it!

The "Edit list" page can now be tested. Example: https://eventmetrics-dev.wmflabs.org/programs/74/events/135/revisions

@MusikAnimal, should I move to Product/QA Review? Or are you still working on it?

In T206495#4776107, @jmatazzoni wrote:

@MusikAnimal, should I move to Product/QA Review? Or are you still working on it?

I wanted to get a design/product sign-off before merging the code. Everything is on https://eventmetrics-dev.wmflabs.org. I don't know which column makes the most sense in this case, since we usually merge the code first (maybe we shouldn't).

@MusikAnimal, this is looking good. I checked everything above off except for two items:

• I'll let @Prtksxna confirm that the fonts/design is correct (though it looks right to me).
• The "View all data" button needs to be renamed "View edit list". (You did just what we said but what we told you was wrong: since we renamed the page, we should go ahead and rename the button, to be consistent in all places.)

When you've completed those, please put back in the QA/Product column

The "View all data" button needs to be renamed "View edit list".

Done. I'll move back to "Needs Review/Feedback" for a design signoff.

@MusikAnimal, I'm bouncing this back to Development for one last thing— sorry to keep adding new requirements! But in the meeting yesterday I realized that if this goes to production ahead of some of the functionality it was designed to accompany we'll have a small UI problem. Namely, a user is on the Event Summary page who asks to "Download reports" from the new button will naturally expect that she will download the Event Summary report. But she won't—she'll get the Edit List report, since it is the only one currently available.

The solution here is to temporarily make the Download Reports button unavailable on this page until we can incorporate some other reports and get the menu working, at which point we'll bring the button back.

You can either put the button into the gray, unavailable state. Or you could just comment it out or whatever to make it disappear. Your call.

In T206495#4782558, @jmatazzoni wrote:

@MusikAnimal, I'm bouncing this back to Development for one last thing— sorry to keep adding new requirements! But in the meeting yesterday I realized that if this goes to production ahead of some of the functionality it was designed to accompany we'll have a small UI problem. Namely, a user is on the Event Summary page who asks to "Download reports" from the new button will naturally expect that she will download the Event Summary report. But she won't—she'll get the Edit List report, since it is the only one currently available.

The solution here is to temporarily make the Download Reports button unavailable on this page until we can incorporate some other reports and get the menu working, at which point we'll bring the button back.

You can either put the button into the gray, unavailable state. Or you could just comment it out or whatever to make it disappear. Your call.

There are still two reports to download, no? I'm not sure current users will be confused, since the options are the same as they always have been. Before they had two separate buttons, and now they're nested in a dropdown. In the future, we'll merely be putting additional links in this dropdown (or replacing existing ones, not sure), right?

If the word "report" is the concern, maybe we can just change the button text to "Download". How does that sound?

In T206495#4783569, @MusikAnimal wrote:

....There are still two reports to download, no? I'm not sure current users will be confused, since the options are the same as they always have been.

They are not the same because in the past there was no report you could download from the Event Summary page. The only report on offer was the Edit List, which you could get only from the Edit List page. Please leave that page just as it is; the dropdown works just fine there. The ask here is to gray out the Download button on the Event Summary page only.

In T206495#4783587, @jmatazzoni wrote:

In T206495#4783569, @MusikAnimal wrote:

....There are still two reports to download, no? I'm not sure current users will be confused, since the options are the same as they always have been.

They are not the same because in the past there was no report you could download from the Event Summary page. The only report on offer was the Edit List, which you could get only from the Edit List page. Please leave that page just as it is; the dropdown works just fine there. The ask here is to gray out the Download button on the Event Summary page only.

Got it, that makes perfect sense. I'm going to go with hiding the dropdown entirely on the Event Summary page, which I think you said was acceptable. That will make it the same as things are now in production.

In T206495#4783987, @MusikAnimal wrote:

...'m going to go with hiding the dropdown entirely on the Event Summary page, which I think you said was acceptable.

Yep, that will be good—just make it so it's easy to bring it back.

@jmatazzoni Done, see for example https://eventmetrics-dev.wmflabs.org/programs/81/events/141

Can this be closed?

In T206495#4829965, @MusikAnimal wrote:

Can this be closed?

I think so but let me go through it tomorrow.