opnsense / core

OPNsense GUI, API and systems backend
https://opnsense.org/
BSD 2-Clause "Simplified" License
3.36k stars 753 forks source link

[Feature Request] Make titles clickable in the Announcements dashboard widget #7958

Open fongd opened 1 month ago

fongd commented 1 month ago

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

Is your feature request related to a problem? Please describe.

It's common for blog/post/forum/etc titles to be clickable to expand its content but in the current OPNsense implementation, you have to click the announcement's timestamp to go to the linked post.

Describe the solution you like

Please link the individual announcement's title to the corresponding forum post.

Describe alternatives you considered

N/A

Additional context

opnsense_announcement

A picture says a thousand words. :) The title in the red square should be a clickable link for each announcement.

fichtner commented 1 month ago

I did change it for exactly the opposite reason. GitHub and social media mostly has the links on the date.

fongd commented 1 month ago

On the GitHub issues list, you click the issue title to "expand" the ticket. On sites like ArsTechnica, TheVerge, etc., you click the title to go to the article. Most RSS readers I've used let you click the title to expand that article. To me, clicking the date means that you want to see all articles posted on the same date, it's not as intuitive to me that you would click the date to see the post rather than the title (which also appears above the date in the Announcements widget so kinda makes sense that it should be clickable).

fichtner commented 1 month ago

On the GitHub issues list you don't read the issue contents. On the widget you do. :)

fichtner commented 1 month ago

Change in question: 929d779b7

To reiterate, the link style was distracting, the click is spurious since you already see the contents. Now the date was added and it was a good opportunity to remove the obvious link that leads to the same information.

I see your point, but I want to avoid to go back and forth because of style changes that will never make more than 50% happy.

fongd commented 3 weeks ago

I don't know what the old link style looked like but there's no reason you couldn't keep the same link style as you have it now (orange bold, no underline) but make the title clickable. Add an underline on hover of the title if you are OK with that. It's clean and easy, and I will point out this is how it works in the OPNsense forums as well, not to mention countless other "list" styles elsewhere.

IMO, what's even more distracting is that the mouse pointer changes to a hand icon when hovering over any of the dashboard widgets, which makes it seem like you can click-to-drag to scroll the contents of said widget even though you can't. I just noticed this behaviour in Safari. But that is another issue so that's all I'll say on that.

Anyway, no need to monopolize your time on this. I understand certain decisions were made and I can respect that.