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.

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

fix size and alignment for frontpage icons #3

Merged
merged 1 commit into from Jun 4, 2014
Merged

fix size and alignment for frontpage icons #3

merged 1 commit into from Jun 4, 2014

Conversation

ghost
Copy link

@ghost ghost commented Jun 4, 2014

The icons in the front page are often a little odd - hoverover makes some pixels displace, the larger images look 'scrunched' when scaled down to ~48px. These icon updates will make browsers properly display without odd artifacts (and reduce size for many of them). It also slightly improves the spacing as a few icons were not properly aligned.

@ghost
Copy link
Author

ghost commented Jun 4, 2014

is this related to #1 ?

Afterster added a commit that referenced this pull request Jun 4, 2014
fix size and alignment for frontpage icons
@Afterster Afterster merged commit b4c0e42 into ampache:master Jun 4, 2014
@ghost ghost deleted the frontpage-icons branch June 4, 2014 07:24
@ghost ghost restored the frontpage-icons branch June 4, 2014 07:24
@Afterster
Copy link
Member

This is far better indeed. Thanks a lot :)
About #1, I'm not sure this is what @SUTJael was looking for, I will let him answer/close about his issue.

@ghost ghost deleted the frontpage-icons branch June 4, 2014 07:24
@SUTJael
Copy link
Contributor

SUTJael commented Jun 4, 2014

#1 is related to icons into the "features" section. We should find more appropriate icons describing the block content (same for donate).

@SUTJael SUTJael mentioned this pull request Jul 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants