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

Portrait Issues #44

Open
XinsongDu opened this issue May 23, 2020 · 6 comments
Open

Portrait Issues #44

XinsongDu opened this issue May 23, 2020 · 6 comments

Comments

@XinsongDu
Copy link
Member

Four people in "People" page (https://lemaslab.github.io/people/) do not have their avatar, please work on asking them for their photos and put the photos on our website.

@thulium1
Copy link
Contributor

thulium1 commented Jun 1, 2020

I have email Andrew, Emily, and Sakina. However, Joshua do not have any contact or any information on the people page, therefore, I don't know how to contact him.

@XinsongDu XinsongDu changed the title Ask People with missing portraits for their photos Portrait Issues Jun 6, 2021
@XinsongDu
Copy link
Member Author

@sheilarodcast Please crop your portrait to 1:1.168 (width:height) and make a pull request. Detailed steps would be:

  1. Merge master branch to sheila branch.
  2. Make changes to the portrait.
  3. Make pull request from sheila branch to master branch.

@sheilarodcast
Copy link
Contributor

sheilarodcast commented Jun 8, 2021 via email

@XinsongDu
Copy link
Member Author

@sheilarodcast your pull request was not appeared on this repo. Where did you make the change? Steps of making changes to the website would be:

  1. merge master branch to your own branch sheila (i.e., make a pull request from master to sheila)
  2. make changes you want (in this situation, it is cropping the portrait to 1:1.618)
  3. make a pull request to the master branch (i.e., make a pull request from sheila to master).
    The pull request is a button on the GitHub repo page:

Screen Shot 2021-06-08 at 2 33 00 PM

@sheilarodcast
Copy link
Contributor

sheilarodcast commented Jun 8, 2021 via email

@XinsongDu
Copy link
Member Author

Oh, I see where you made the change (https://github.com/sheilarodcast/lemaslab.github.io/blob/master/images/people/Sheila_Rodriguez.jpeg). The modification is great.

Since there is already a branch for you in this repo (https://github.com/lemaslab/lemaslab.github.io/tree/sheila), you need to work using this link under lemaslab repo instead of a forked repo. Working on a forked repo will not make any change to the website.

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

No branches or pull requests

3 participants