v2.ocaml.org icon indicating copy to clipboard operation
v2.ocaml.org copied to clipboard

Different button colors on different pages

Open dinakajoy opened this issue 3 years ago • 25 comments

I just noticed that the button color on other pages are green while it's grey on news page. Accessibility wise, I think it needs to be fixed. Because from the home page down, users already have it registered in their mind that green means it's a button.

Button on other pages green

Button on news page grey

I think it will create better user experience. If it's a good fix, then I would have it fixed.

dinakajoy avatar Apr 01 '21 10:04 dinakajoy

@dinakajoy I truly feel the coloring of the read more buttons is misleading. It initially struck me as a disabled button till I clicked on it.

Although I just noticed that the linking of the "Read More" button is not the same as the title to the articles they are attached to. Did you notice this?

lindaokorie27 avatar Apr 02 '21 17:04 lindaokorie27

Nothing has been said yet by the maintainers on it. Regarding the link, can you drop a screenshot please because it was okay when I tried it except when you try to open in new tab.

dinakajoy avatar Apr 02 '21 17:04 dinakajoy

I think the 'read more' button on the news can be improved. It is an accordion not a button, so I suggest we indicate a drop down arrow with 'read more' text so as not to confuse users.
When 'Read More' is clicked, it opens up the remaining content and still shows other news topic after it which is confusing as users think they are on a new page not the current page.

dinakajoy avatar Apr 07 '21 09:04 dinakajoy

@dinakajoy Personally I don't mind the button background color though there is no harm in changing it to be consistent with other buttons as well. What concerns me most is its positioning. Someone reading through will attempt to click the ... after encountering it. I believe it would make a lot more sense to place it immediately after ... to convey to the user there is a hidden content which can be revealed by clicking the button.

nibble0101 avatar Apr 07 '21 10:04 nibble0101

@dinakajoy , thanks for filing the issue! I agree that there's room for improvement, we just need to discuss the details. It should be implemented by someone who hasn't worked on 2 or more simple issue yet though :)

pitag-ha avatar Apr 07 '21 10:04 pitag-ha

@nibble0101 if the read more button is placed immediately after the ... then it would be better it is styled more as a link.

lindaokorie27 avatar Apr 07 '21 10:04 lindaokorie27

hey @pitag-ha , till now I have submitted 2 PRs. one for a good-first-issue https://github.com/ocaml/ocaml.org/issues/1258 and improved contribution.md. can I work on this issue, as it will be my 2nd good-first issue? I'm also observing the site carefully and after this I'll try and open an issue with medium difficulty to work on :))

yashi-hub avatar Apr 07 '21 11:04 yashi-hub

Hi @pitag-ha , I would like to work on this issue as my 1st good-first issue.

oma16 avatar Apr 07 '21 11:04 oma16

@yashi-hub, it's nice that you're eager to contribute, but every applicant can only work on one issue tagged as good-first-issue.

@oma16, sure, go ahead :)

pitag-ha avatar Apr 07 '21 13:04 pitag-ha

@pitag-ha Thanks ,am glad.

oma16 avatar Apr 07 '21 14:04 oma16

Screenshot 2021-04-07 151940 @pitag-ha The button before Screenshot 2021-04-07 153715 Screenshot 2021-04-07 153905 the after

oma16 avatar Apr 07 '21 14:04 oma16

Thanks @oma16, I'm not sure if I can see well the before and after screenshots, but I think that's already better! Have you also tried indicating a drop down arrow with 'read more' as suggested by @dinakajoy and/or making the extension points clickable as pointed out by @nibble0101 might be what users expect? I'm not sure which solution would be best.

pitag-ha avatar Apr 07 '21 15:04 pitag-ha

Although I just noticed that the linking of the "Read More" button is not the same as the title to the articles they are attached to. Did you notice this?

@LindieK , I hadn't noticed that yet. Could you point me to an article where that happens?

pitag-ha avatar Apr 07 '21 15:04 pitag-ha

Thanks @oma16, I'm not sure if I can see well the before and after screenshots, but I think that's already better! Have you also tried indicating a drop down arrow with 'read more' as suggested by @dinakajoy and/or making the extension points clickable as pointed out by @nibble0101 might be what users expect? I'm not sure which solution would be best.

@pitag-ha, I will try it now .

oma16 avatar Apr 07 '21 15:04 oma16

@pitag-ha I discovered that the read more button reveals the rest of the article more like an accordion and clicking the title of the article takes you an article on the same topic on an external site. I realize that this might be expected behaviour as all the other articles listed behaved the same way.

lindaokorie27 avatar Apr 07 '21 17:04 lindaokorie27

@pitag-ha, I have tried the drop down arrow and this is the output Screenshot 2021-04-07 184054

oma16 avatar Apr 07 '21 17:04 oma16

I would say that if we went down that route that would have to be styled differently. But I'm no design expert at all :) You can try out different things and see what works. If any of the others also has some more input, that's also appreciated :)

pitag-ha avatar Apr 07 '21 18:04 pitag-ha

I have seen the v-shape symbol being used to symbolize the presence of a collapsible content but I think it looks funny here with the preceding .... And I think you also need to think about reversing the symbol if the accordion is open to demonstrate the reverse effect.

nibble0101 avatar Apr 07 '21 18:04 nibble0101

I have seen the v-shape symbol being used to symbolize the presence of a collapsible content but I think it looks funny here with the preceding .... And I think you also need to think about reversing the symbol if the accordion is open to demonstrate the reverse effect.

Screenshot 2021-04-07 201139

Screenshot 2021-04-07 201232

oma16 avatar Apr 07 '21 19:04 oma16

I think the ellipses is enough indication that there is more ahead. Even screen readers would pick that up. Based on existing conventions, the text "Read more" always signifies there is more ahead on whatever preceded it.

lindaokorie27 avatar Apr 07 '21 19:04 lindaokorie27

I would say that if we went down that route that would have to be styled differently. But I'm no design expert at all :) You can try out different things and see what works. If any of the others also has some more input, that's also appreciated :)

Screenshot 2021-04-08 115015

Screenshot 2021-04-08 114101

oma16 avatar Apr 08 '21 10:04 oma16

Hi @oma16! I think the screenshot you've provided here: https://github.com/ocaml/ocaml.org/issues/1353#issuecomment-815662944 is good and conveys intent of the buttons well. You can go ahead and open a PR for this change. Thanks!

gs0510 avatar Apr 08 '21 12:04 gs0510

Hello @gs0510 , I will do that now. Thanks.

oma16 avatar Apr 08 '21 13:04 oma16

i also want to contribute

harshsingh941 avatar Feb 08 '22 12:02 harshsingh941

@dinakajoy can ou assign this to me if it's still open?

vishalsingh2972 avatar Apr 28 '23 19:04 vishalsingh2972