UNSWComputing / rUNSWift-docs

rUNSWift docs
https://runswift.readthedocs.io/
2 stars 3 forks source link

update team photo #38

Closed abhivij closed 1 year ago

abhivij commented 1 year ago

suggesting alternate team photo including just the members

pzrq commented 1 year ago

Builds on top of #31

ijnek commented 1 year ago

Looks good, make sure you delete the .png photo!

pzrq commented 1 year ago

Looks good, make sure you delete the .png photo!

Yeah if it doesn't happen in this PR someone probably will raise another one.

pzrq commented 1 year ago

To be clear, just waiting on someone else to merge, so it represents a travelling team majority decision, even if it's not necessarily unanimous.

abhivij commented 1 year ago

@pzrq @ijnek removed unused photo

n33r4j commented 1 year ago

👍

n33r4j commented 1 year ago

A recruit reported that the photo was not displaying correctly on Safari. Can someone with a Mac check it out ?

jayenashar commented 1 year ago

image it would be better cropped but it seems fine to me

pzrq commented 1 year ago

@n33r4j Please ask for a screenshot in future, or better yet, let them know PRs are welcome too!

e.g. Safari could also be iOS (iPhone SE 2 below) or iPadOS (not shown).

I also cannot reproduce anything obviously wrong?

Here's Firefox 110.0 vs Safari 16.3 on an Intel Mac (not sure about M1/M2 though it should be the same MacOS / Safari):

Screenshot 2023-02-22 at 21 40 12

iPhone SE 2 / Safari 16.3: IMG_9FCD9AFE5B27-1