Harmony feedback

phenomlab

Unpopular opinions can be useful too

That's on the provision that they are presented in a way that doesn't cause unnecessary offence, or worded constructively. Case arguments are always useful rather than simply forcing a view without considering that this is still at RC stage.

<baris>

@phenomlab there are lots of people out there, some will give feedback very bluntly without thinking if it's gonna hurt someone's feelings. It doesn't mean their feedback is right or wrong, or less valuable. We discuss it openly and internally and decide if it's worth taking into account.

In @brazzerstop's case we agreed on the teasers. The other points not so much.

In the end everyone should be able to post their ideas freely and let the community decide if it's worth listening to.

phenomlab

@baris agreed, but there is a way to convey that message whilst still using an appropriate tone. The original message comes across as less than constructive in my view, but the jury may still be out on that.

julian

I think tone is important as well, when conveying feedback. When responding to others in person or on the internet, there's a baseline level of respect that should be afforded.

While I am certain @brazzerstop's intention wasn't one of malice, I felt that it demonstrated a simple misassumption on their part, one that is common to all open source software: that their feedback is better than no feedback at all; that it should be valued because they spent the time to offer it, and should be respected for that reason.

The truth of it is that @brazzerstop didn't pay a cent to NodeBB to use the software (which is great, in a way!), but it also comes with a very important caveat:

THERE IS NO WARRANTY FOR THE PROGRAM, TO THE EXTENT PERMITTED BY APPLICABLE LAW. EXCEPT WHEN OTHERWISE STATED IN WRITING THE COPYRIGHT HOLDERS AND/OR OTHER PARTIES PROVIDE THE PROGRAM "AS IS" WITHOUT WARRANTY OF ANY KIND, EITHER EXPRESSED OR IMPLIED, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE. THE ENTIRE RISK AS TO THE QUALITY AND PERFORMANCE OF THE PROGRAM IS WITH YOU. SHOULD THE PROGRAM PROVE DEFECTIVE, YOU ASSUME THE COST OF
ALL NECESSARY SERVICING, REPAIR OR CORRECTION.

@brazzerstop is free to offer his opinion. I am free to use it, respond it, or ignore it.

In this case I have chosen to take his criticisms objectively, but his wording has coloured future interactions with his posts.

Adam C

The hyperlink on the message box is hard to read.

julian

@Adam-C I think you want this thread

Adam C

Thank you. @julian I realize now that I should have posted my comment in that thread. However, I find that the thread has become quite lengthy, and difficult to find what I look for. Perhaps creating a sub-category with separate threads for each issue, or even using Github Issues, would be a better approach.

Regarding the issue I had, it appears to be a simple fix by adding the "alert-link" class. As per the documentation, "Use the .alert-link utility class to quickly provide matching colored links within any alert."

julian

@Adam-C granted you can also open a GitHub issue. This is one we can fix.

sweetp

What’s stopping ppl from using a different theme? The old v2 default is still available right?

For what it’s worth, I like the new look of Harmony. The small things I don’t like I’ve changed with my own custom theme.

phenomlab

@sweetp exactly. Great points. And I, like yourself have made changes to suit my tastes. Personally, I quite like being able to see the last response to a topic without having to open it.

julian

@sweetp that's correct, the Persona theme is compatible with v3. Many of our clients are running Persona, or derivatives of it, and so it would be unwise to spring a new theme on them

brazzerstop

@baris I also copied this settings here to hide the preview of the topics on the category page Thank you!

https://github.com/NodeBB/nodebb-theme-harmony/blob/main/templates/partials/categories/item.tpl#L57

<div component="topic/teaser" class="teaser col-lg-6 {{{ if !config.theme.mobileTopicTeasers }}}d-none d-lg-block{{{ end }}}">

<baris>

@brazzerstop Ahh I forgot that for /categories page, added that into harmony as well.