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

feat: slightly adjusted palm trees to prevent clipping in Bangkok's Atrium #240

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dribbleondo
Copy link
Contributor

@dribbleondo dribbleondo commented Nov 22, 2024

Adjusts some palm tree's in the atrium in Bangkok to not clip so heavily with the flower boxes they are planted in.

About five of these are clipping heavily into the flowerboxes.

BEFORE:
screenshot_8184

AFTER:
image

fixes #239

@VoodooHillbilly
Copy link
Contributor

There was still some clipping when the tree sways for f2bb1bc577dbe9e4.
It looks like 6d44965dce15fc4b is slightly clipping, too.

@dribbleondo
Copy link
Contributor Author

dribbleondo commented Dec 5, 2024

There's going to be some clipping regardless of what I do due to where they are positioned (plant clipping was also a concern), I just didn't want the tree's to literally be in concrete.

@VoodooHillbilly
Copy link
Contributor

Leaves and smaller plants clipping is inevitable. Clipping with the concrete box is what I was referring to on both of those trees. It's not severe, but I thought I'd mention it since you're already moving them.

clipping

@dribbleondo
Copy link
Contributor Author

dribbleondo commented Dec 8, 2024

Leaves and smaller plants clipping is inevitable. Clipping with the concrete box is what I was referring to on both of those trees. It's not severe, but I thought I'd mention it since you're already moving them.

@VoodooHillbilly Turns out I hadn't adjusted that tree, so I added it to the new commit!

@dribbleondo
Copy link
Contributor Author

LGTM!

@VoodooHillbilly
Copy link
Contributor

VoodooHillbilly commented Dec 17, 2024

Screenshots of the two edited trees (6d44965dce15fc4b and f2bb1bc577dbe9e4) that are still clipping with the flower box.

6d44965dce15fc4b clipping

f2bb1bc577dbe9e4 clipping

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.

Bangkok Atrium Trees are Heavily Clipping into the Flowerboxes
2 participants