search

the-hideout / tarkov-dev

The official site for tarkov.dev - A web app to track item prices, view trader barters, quests, maps, and much more!
https://tarkov.dev
MIT License
130 stars 48 forks source link

Make ammo chart height dynamic #913

Closed Razzmatazzz closed 2 months ago

Razzmatazzz commented 2 months ago

When additional calibers are added, the bottom of the ammo chart legend gets cut off. This should make the height dynamic depending on how many different calibers there are.

Resolves #908

github-actions[bot] commented 2 months ago

👋 Thanks for opening a pull request!

If you are new, please check out the trimmed down summary of our deployment process below:

  1. 👀 Observe the CI jobs and tests to ensure they are passing

  2. ✔️ Obtain an approval/review on this pull request

  3. 🚀 Deploy your pull request to the development environment with .deploy to development

  4. 🚀 Deploy your pull request to the production environment with .deploy

    If anything goes wrong, rollback with .deploy main

  5. 🎉 Merge!

Note: If you have a larger change and want to block deployments, you can run .lock --reason <reason> to lock all other deployments (remove with .unlock)

You can view the branch deploy usage guide for additional information

Razzmatazzz commented 2 months ago

.deploy

github-actions[bot] commented 2 months ago

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to production

You can watch the progress here 🔗

Branch: dynamic-ammo-chart-height

github-actions[bot] commented 2 months ago

Deployment Results ✅

Razzmatazzz successfully deployed branch dynamic-ammo-chart-height to production

Environment URL: tarkov.dev