Harmony feedback
-
-
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.
-
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."
-
@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 }}}">
-
@brazzerstop Ahh I forgot that for /categories page, added that into harmony as well.