Small Updates / Quick Wins #228 #229
Conversation
ghost
requested a review
from 
ghost
mentioned this pull request
ghost
added
T25m
iteles
added
the
in-review
views/index.html
Outdated
| @@ -1,15 +1,15 @@ | |||
| <div class="pure-g full-height-block dark-bg splash"> | |||
| <a href="/team"><div class="splash-container"></div></a> | |||
| <div class="pure-u-1 splash-tagline"> | |||
| <span>dwyl is a <span class="tagline-highlight">team</span> of people using <span class="tagline-highlight">technology</span> to solve real problems</span> | |||
| <span>dwyl /dwɪɫ/ is a <span class="tagline-highlight">worldwide community</span> of people using <span class="tagline-highlight">technology</span> to solve real problems.</span> | |||
There was a problem hiding this comment.
Does anyone really understand that phonetic thing?
There was a problem hiding this comment.
I do...not sure if other people do...this comes from dwyl/video#24 and dwyl/hq#249 - feel free to remove if we don't want it on there!
There was a problem hiding this comment.
@markwilliamfirth I'll leave it to you to decide whether this looks confusing or not:
| </p> | ||
| </div> | ||
| </div> | ||
| <div class="pure-u-1 pure-u-md-1-3 flex-center section-box-wrapper"> | ||
| <div class="section-box"> | ||
| <div class="offering-icon user-testing-icon"> | ||
| </div> | ||
| <h3>User testing</h3> | ||
| <h3>Co-working</h3> |
There was a problem hiding this comment.
I don't mind this change given it will only be here for two weeks, but the importance of user testing needs to be highlighted in our new content.
Good to get this on here though 👍
There was a problem hiding this comment.
I thought it was against our values to use the word "user" 😆 mentioned "human-centred design" in it's place in training bit
There was a problem hiding this comment.
😊 User testing is a well-known term.
There was a problem hiding this comment.
Think SEO on this one. We just don't refer to our community as users: https://github.com/dwyl/start-here/blob/master/manifesto.md#we-do-not-have-users-we-have-people
views/index.html
Outdated
| </p> | ||
| <p> | ||
| <i>We fully support the <a href="http://www.chagossupport.org.uk/">Chagossians</a> and their right to claim profits from the sale of .io domains associated with their homeland.</i> | ||
| Copyright 2016 © DWYL LTD, company number 09525434 registered in England & Wales at 104 Mary Datchelor Close, London SE5 7DY, United Kingdom. All rights reserved. |
There was a problem hiding this comment.
Can do next time! As I said there are lots more changes to make 😛
views/index.html
Outdated
| <p> | ||
| We provide professional training for developers in TDD, Node.js and React. | ||
| Hands-on, immersive learning designed to enlighten, inspire and empower. Workshops in agile development, product ownership, human-centered design and more. |
There was a problem hiding this comment.
Did you run the code to make sure that having a way bigger caption for this picture than the others won't make it look weird or break the styling?
There was a problem hiding this comment.
Didn't notice that - does it do that?
There was a problem hiding this comment.
Took me 2 mins to pull this branch, npm i && npm start. Always worth running the code you're changing.
There was a problem hiding this comment.
Do you want them all the same size?
There was a problem hiding this comment.
No... I just want it to not look ugly ;)
There was a problem hiding this comment.
Do I have to push another commit to this PR? Not sure how to do this...never done this before...
There was a problem hiding this comment.
You went through Founders & Coders, of course you have! 😆
Yea, just push another commit to this branch and it will show up in this PR.
iteles
removed
awaiting-review
|
|
#228
Minor updates to site:
This is not a complete update - there are still many changes that need doing, these are just minor ones that I spotted as I was updating Anita's team status