Skip to content
This repository has been archived by the owner on Oct 14, 2021. It is now read-only.

Change officer profile element from div to image tag #128

Open
erikvanderwerf opened this issue Aug 7, 2018 · 1 comment
Open

Change officer profile element from div to image tag #128

erikvanderwerf opened this issue Aug 7, 2018 · 1 comment
Assignees
Labels

Comments

@erikvanderwerf
Copy link
Member

Currently profiles are not selectable in a traditional sense. Users can't download/open in new tab.

@erikvanderwerf erikvanderwerf self-assigned this Aug 7, 2018
@mattmolo
Copy link
Member

mattmolo commented Aug 7, 2018

The main reason for this is because we got pictures of all sorts and sizes. Using some css background tricks it would center the picture and make sure to proportionally fill out the div. There may be a better way to do this now with image tags or something, but this was an easy way to do it at the time. I don't know if we actually do care for the use case of downloading officer images..

You could also resize and crop everything yourself... manually or with like ImageMagick.. I just made tried to make it easier for anyone else uploading images

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants