search

NeuroGrid / folk-on-the-hill

For the Harrow Folk Festival
MIT License
0 stars 2 forks source link

client would like image size adjusted so that more can be seen of each artist #38

Open tansaku opened 5 years ago

tansaku commented 5 years ago

client is happy for images to be much taller

tansaku commented 5 years ago

particularly at some resolutions some of the current images are not working so well:

javpet commented 5 years ago

As I see it's basically a tablet issue so 400px < x < 990px, sure it's no problem to make them higher. If we lack of the tablet design we can only guess how it should look. :/

tansaku commented 5 years ago

hmm, we can just do our best :-)

Client also asking similar for venue images - just uploading a new one

javpet commented 5 years ago

Fixes #38 https://github.com/NeuroGrid/folk-on-the-hill/pull/40

tansaku commented 5 years ago

thanks @javpet - note I don't know that it make much sense to put

Fixes #38 #40

here - I think we want the fixes in the PR so that they close the issue when we merge them, do you know what I mean?

javpet commented 5 years ago

Yep absolutely!

On Fri, 1 Mar 2019 at 20:38, Sam Joseph notifications@github.com wrote:

thanks @javpet https://github.com/javpet - note I don't know that it make much sense to put

Fixes #38 https://github.com/NeuroGrid/folk-on-the-hill/issues/38 #40 https://github.com/NeuroGrid/folk-on-the-hill/pull/40

here - I think we want the fixes in the PR so that they close the issue when we merge them, do you know what I mean?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/NeuroGrid/folk-on-the-hill/issues/38#issuecomment-468786169, or mute the thread https://github.com/notifications/unsubscribe-auth/AI5vdhCjHv_7KAxXO31gwIrBDR6FJb_nks5vSYG0gaJpZM4bS5_M .