Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Welcome variation | About us image isn't great and doesn't work above 1366x768 px #50

Closed
iteles opened this issue Oct 30, 2015 · 4 comments
Closed

Welcome variation | About us image isn't great and doesn't work above 1366x768 px #50

iteles opened this issue Oct 30, 2015 · 4 comments
Labels
bug Suspected or confirmed bug (defect) in the code duplicate Duplicate of an existing issue priority-4 Deprioritised until all other higher priority items are complete.

Comments

@iteles
Copy link
Member

iteles commented Oct 30, 2015

Might not be too much of an issue in terms of a CSS fix as I think we need to change this picture out pretty quickly. But the issue at the moment is as follows.

1366 x 768px:
screen shot 2015-10-29 at 22 58 46

1920 x 1080px, cuts off people's heads:
screen shot 2015-10-29 at 22 59 03
@iteles iteles added bug Suspected or confirmed bug (defect) in the code priority-4 Deprioritised until all other higher priority items are complete. labels Oct 30, 2015
@nelsonic
Copy link
Member

nelsonic commented Nov 1, 2015

We have a betterer image: #51 (comment) would you not agree...? 😉

@iteles
Copy link
Member Author

iteles commented Nov 1, 2015

👍 I originally used this one as a test image because it's long and thin - the one you mention doesn't lend itself as well to a thin pane of information which is what I had in mind at the time.

I tried the bridge photo too, but it didn't work as well.

@nelsonic
Copy link
Member

nelsonic commented Nov 1, 2015

Sounds like you might have a fun task on your hands... #53

@iteles iteles added the duplicate Duplicate of an existing issue label Nov 1, 2015
@iteles
Copy link
Member Author

iteles commented Nov 1, 2015

Now null and void given that we're completely changing the website ( #51 ) and #53 exists.

@iteles iteles closed this as completed Nov 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Suspected or confirmed bug (defect) in the code duplicate Duplicate of an existing issue priority-4 Deprioritised until all other higher priority items are complete.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nelsonic @iteles