servers.coop
/
servers.coop
1
0
Fork 0
Code Issues 2 Pull Requests Projects Releases Wiki Activity

add contact page #10

Closed
knoflook wants to merge 3 commits from dev into main
pull from: dev
merge into: servers.coop:main
Conversation 6 Commits 3 Files Changed 2 +84
knoflook commented 2022-04-04 13:57:15 +00:00
Owner
Copy Link

it's not great but i'm not a designer 😆 closes #8

it's not great but i'm not a designer 😆 closes #8
knoflook added 1 commit 2022-04-04 13:57:18 +00:00
knoflook added 1 commit 2022-04-04 14:04:25 +00:00
3wordchant commented 2022-04-06 14:18:15 +00:00
Owner
Copy Link

Looks good, only suggestion is to change the "Get in touch" link in the intro to go to the new page as well?

image

Looks good, only suggestion is to change the "Get in touch" link in the intro to go to the new page as well? ![image](/attachments/21dca585-e29d-4744-84db-b99d93d80323)
image.png
22 KiB
3wordchant commented 2022-04-11 22:29:08 +00:00
Owner
Copy Link

@knoflook you were right, I was wrong, it looks fine YOLO'd onto the main homepage at the bottom. I put my changes in a new 3wc-contact-section branch -- should I make a separate PR, or combine with this one? i.e. do you think it works?

Also yeah still down to change the "get in touch" link to point to the anchor.

@knoflook you were right, I was wrong, it looks fine YOLO'd onto the main homepage at the bottom. I put my changes in a new `3wc-contact-section` branch -- should I make a separate PR, or combine with this one? i.e. do you think it works? Also yeah still down to change the "get in touch" link to point to the anchor.
image.png
73 KiB
image.png
77 KiB
image.png
78 KiB
Screenshot from 2022-04-11 23-43-14.png
74 KiB
image.png image.png image.png Screenshot from 2022-04-11 23-43-14.png
knoflook commented 2022-04-12 12:08:12 +00:00
Author
Owner
Copy Link

I don't like the way it looks when moved to the index page, it's lacking an illustration and the beige section gets too big, I'd put it in a different page or ask yusf to doodle something maybe? I'd also like to have the buttons arranged in rows like so:

[ join our matrix space ] [ join our matrix group ]
		[ join our IRC ]
[ join our mailing list ] [ send us a mail ]

As for the "Get in touch" button I think we can drop it completely

I don't like the way it looks when moved to the index page, it's lacking an illustration and the beige section gets too big, I'd put it in a different page or ask yusf to doodle something maybe? I'd also like to have the buttons arranged in rows like so: ``` [ join our matrix space ] [ join our matrix group ] [ join our IRC ] [ join our mailing list ] [ send us a mail ] ``` As for the "Get in touch" button I think we can drop it completely
knoflook added 1 commit 2022-04-12 12:37:55 +00:00
knoflook commented 2022-04-12 12:41:40 +00:00
Author
Owner
Copy Link

i moved the buttons so they are in rows, but i still think it's not looking great.

i moved the buttons so they are in rows, but i still think it's not looking great.
3wordchant commented 2022-04-12 12:43:22 +00:00
Owner
Copy Link

@knoflook yeah I agree, it would be nice to have another illustration, I wonder if we should wait until Yusf finishes the new designs, in case he ends up wanting to redo the website in general?

Fine with keeping the separate until then if you prefer.

Agreed that the buttons in rows looks way nicer 🖌️

@knoflook yeah I agree, it would be nice to have another illustration, I wonder if we should wait until Yusf finishes the new designs, in case he ends up wanting to redo the website in general? Fine with keeping the separate until then if you prefer. Agreed that the buttons in rows looks way nicer :paintbrush:
knoflook commented 2022-04-14 14:00:18 +00:00
Author
Owner
Copy Link

I guess i'll just merge it, we're overthinking it. it's not like the site is getting huge traffic anyway

I guess i'll just merge it, we're overthinking it. it's not like the site is getting huge traffic anyway
knoflook force-pushed dev from 3e40c5a57d to e5d3d28c54 2022-04-14 14:00:40 +00:00 Compare
knoflook closed this pull request 2022-04-14 14:03:35 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: servers.coop/servers.coop#10
No description provided.
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?