-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added pages for "Contact" and "About" #128
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The contact details are also listed on the Contribute
page so if we introduce a special Contact
page, we should remove the info from there and link to the contact page instead.
For the About
page I would say that this kind of information is partially covered by the main page. It also contains some words about the project...
I disagree, the Contribute page lists these things for community participation.
As I already said in #98, it does not cover what this should really be about. |
But the means of contact are the same. Thus you'd only be duplicating the information which is never a good idea.
That's why I wrote partially ☝️ |
No.
It is done very often.
I saw that, but it seemed, that you used it as an argument against an about page. |
There no reason why these two ways of contacting can't live on the same page.
It is indeed. But it is never good to do that.
But it has to be done and especially: remembered. That's the issue |
There is, those are two different categories:
I don't see why and how that can be on the same page, also because there are different contents for both. |
They don't differ as we as a team are also reachable on the e.g. IRC. Thus effectively there is no difference. Any user can choose what they think is an appropriate means of communicating with us. |
Well you mean a different level than I do, but instead of continuing this... What would be your solution? Another idea would be some kind of template that could be included, like the download templates. This way you would have the list in one place. |
My idea is to summarize all means of contacting in the
Definitely a lot better than duplicating the information in source. However I think it is better to link to a contact page instead. I mean that's kinda what links are for 🤷 |
If thats what you want; I will at least add a seperate category to the Contact page to seperate between What about the About page now? |
Sounds good.
Well idk what I would want to put there since I don't think this to be vital in the first place. But then again I never look at About pages of other projects (as long as I find information on the actual software I am interested in elsewhere) 🤷 |
I have moved Contact to the first place in the footer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suggestion: It would be nice if the commit description and PR description would mention the menu being moved, and reasoning for it. (Space)
That’s a significant change of existing concept/content.
Thought for later: I think we have an opportunity to implement clickable and readable links and should do so eventually. I presume you put it in like this so they are not easily discoverable by bots/crawlers. Which is fine for now.
hugo/content/about.md
Outdated
| Davide Beatrici | [davidebeatrici](https://github.com/davidebeatrici) | Core developer | | ||
| Robert Adam | [Krzmbrzl](https://github.com/Krzmbrzl) | Core developer, GitHub Management | | ||
| Nik Reist | [ZeroAbility](https://github.com/ZeroAbility) | Release Management | | ||
| Jan Klass | [Kissaki](https://github.com/Kissaki) | Community work | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel undervalued. Is this descriptive of what role I have here and what I have contributed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel this should represents the current situation (that's also why Thorvald isn't listed at all).
Nonetheless you are free to suggest a different role description here. This was just my original suggestion and since you did not answer on my ping on this topic, we left it at that 🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whatever
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Whatever" is not really a good answer. Maybe other team members want a different description etc. as well. I will wait for the results of your team meeting, thus will unresolve this.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, nice.
I mark some small cleanups. I do not consider them blockers, but would like them resolved still.
For the commit message:
Please use present tense for commit messages. "Add pages" instead of "Added pages".
Do not end commit title with a fullstop (it’s not a sentence but a title).
I tried to find a definite reference for this, but have difficulty finding it right now. Anyway, present tense is the convention, starting from the Git project itself.
Otherwise LGTM.
Ok, will upload a fix for that. Some last questions:
|
|
I added the suggestions by @Kissaki. I also changed the headers. Regarding the questions:
|
The commit is now present tense instead of past, but still does not follow the recommended format.
To follow convention it should be
Think of it not as describing what the commit does, but describing what the change itself does. The change is the subject. The change that is applied to source code. And it is a description of what that change does. I feel like this explanation sucks. Anyway, I don’t have anything better right now. |
Add "About" to the menu and move "Sponsors" to footer. Add "Contact" to footer. Signed-off-by: Tobias Gerold <[email protected]> Co-authored by: Robert Adam <[email protected]> Co-authored by: Jan Klass <[email protected]>
Nah I don't like that. People will find that out simply by clicking the link. And it is not like they are automatically registering anywhere just by clicking the link.
Personally I am not interested in this kind of stuff when I look at such a page. If you (or someone else) think that it should be done though, I don't have any objections to adding such a section to the bottom. |
Thank you for your contribution :) |
-Also moves "Sponsors" to the footer menu
The website is missing some general content most websites offer.
It also makes you a bit more trust-worthy and transparent.
Some of it might even be a legal requirement.
Fixes #126 and #98.