Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Better wording for the show comments functionality. #2618

Closed
wants to merge 1 commit into from

Conversation

dwt
Copy link
Collaborator

@dwt dwt commented Aug 18, 2016

User testing for S1 has shown that people miss the fact that they can show the comments column because they think that this is just an indicator how many comments are available. (Reasoning: It looks the same as other comment indicators, but it behaves different).

Thus the resolution: Make it look different too, to give people a reason to suspect that it does something. :)

User testing for S1 has shown that people miss the fact that they can show the comments column because they think that this is just an indicator how many comments are available. (Reasoning: It looks the same as other comment indicators, but it behaves different).

Thus the resolution: Make it look different too, to give people a reason to suspect that it does something. :)
@dwt
Copy link
Collaborator Author

dwt commented Aug 19, 2016

When considering to merge, you guys might want to consider to extend this change to the other adh projects. I didn't feel empowered to decide this on my own.

@xi xi self-assigned this Aug 22, 2016
@xi
Copy link
Collaborator

xi commented Aug 22, 2016

When considering to merge, you guys might want to consider to extend this change to the other adh projects. I didn't feel empowered to decide this on my own.

You already implemented this for almost everything, right??

@xi
Copy link
Collaborator

xi commented Aug 22, 2016

image

I talked to our designer and we agreed that this is not the best approach. The link you changed is primarily information. It is only a link because -- why not? The primary way to get to the comments should be the icon in the bottom right. So we should change that instead.

@xi xi removed their assignment Aug 22, 2016
@dwt
Copy link
Collaborator Author

dwt commented Aug 25, 2016

Well, I don't mind which solution is chosen - but right now the comments functionality is not self explaining and discoverable at all.

I'd say go with the text based change for now, till a better solution is built and then reverse this (smallish) change.

@der-john
Copy link

der-john commented Aug 25, 2016

I'm sorry, @dwt, but I don't think we all approve. I just talked to Petra and she agreed with Nico and @xi, partly because the link isn't just for showing comments, but for writing them and reporting them, too.

But I positively agree with you that we should take your user testing into consideration. In his most recent clickdummy on meinberlin's design, Nico once again suggested to list comments right underneath the proposal itself, cf. https://marvelapp.com/1gc2d4i#14210116. (Furthermore, I believe in @stadine's design there was a call-to-action button saying "Comment on this proposal" inside the resource header.)

I would take your case as an argument to relocate the comments asap.

@dwt dwt closed this Aug 31, 2016
@xi xi mentioned this pull request Sep 12, 2016
@joka joka deleted the 2016-08-mh-comment-better-wording branch February 3, 2018 22:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants