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

[MIRROR] Paintings update: Curators get a cut on patronage + zoom in/out buttons on UI #2066

Merged
merged 1 commit into from
Feb 20, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Mirrored on Nova: NovaSector/NovaSector#1057
Original PR: tgstation/tgstation#81500

About The Pull Request

(Roundstart) Curators now get a 22.5% cut on credits spent on painting patronages (divided by the number of curators). The service department also gets another, 12.5% cut.

This PR also adds zoom in/out buttons to the painting canvas UI. So you don't have to stare at a blob of such enormous squares while the UI is open, which is only good when drawing.

Screenshot copypaste in paint:
it works

Why It's Good For The Game

The painting feature is mostly an end in itself, which is totally fine. I've put quite a few quality-of-life changes into it through the years, and I still want to kick in some stuff. However, I think the curator should actually benefit from them in a more "mechanical" way. Furthermore, I personally prefer them over the random written crap that players make.

Also, as I said above, the canvas UI can feel a tad too big at times.

Changelog

🆑 Ghommie
add: Curators now get a 22.5% cut on credits spent on painting patronages (divided by the number of curators). The service department also gets another, 12.5% cut.
qol: Added a zoom in and out buttons to the painting UI.
/:cl:

…out buttons on UI (#1057)

* Paintings update: Curators get a cut on patronage + zoom in/out buttons on UI (#81500)

## About The Pull Request
(Roundstart) Curators now get a 22.5% cut on credits spent on painting
patronages (divided by the number of curators). The service department
also gets another, 12.5% cut.

This PR also adds zoom in/out buttons to the painting canvas UI. So you
don't have to stare at a blob of such enormous squares while the UI is
open, which is only good when drawing.

Screenshot copypaste in paint:

## Why It's Good For The Game
The painting feature is mostly an end in itself, which is totally fine.
I've put quite a few quality-of-life changes into it through the years,
and I still want to kick in some stuff. However, I think the curator
should actually benefit from them in a more "mechanical" way.
Furthermore, I personally prefer them over the random written crap that
players make.

Also, as I said above, the canvas UI can feel a tad too big at times.

* Paintings update: Curators get a cut on patronage + zoom in/out buttons on UI

---------

Co-authored-by: Ghom <[email protected]>
@Iajret Iajret merged commit c2bff1e into master Feb 20, 2024
24 checks passed
@Iajret Iajret deleted the upstream-mirror-1057 branch February 20, 2024 12:00
AnywayFarus added a commit that referenced this pull request Feb 20, 2024
Iajret pushed a commit that referenced this pull request Apr 19, 2024
* Adds text to lootpanel items [no gbp] (#82722)

## About The Pull Request
Just some qol, this captures a small portion of the item name and fixes
some icon clipping issues

I clipped it at 5 because monke
![Screenshot 2024-04-17
034211](https://github.com/tgstation/tgstation/assets/42397676/64d9ce18-3722-4382-a58d-f58eb6b9f26c)


![gXR2XjslzR](https://github.com/tgstation/tgstation/assets/42397676/3fc93110-e35e-434a-b63b-89aa669090e7)
## Why It's Good For The Game
Handy if youre in a rush and won't upgrade byond to fix the issue
## Changelog
:cl:
fix: Lootpanel additions: Condensed item names for the quick of draw
/:cl:

* Adds text to lootpanel items [no gbp]

---------

Co-authored-by: Jeremiah <[email protected]>
ReezeBL pushed a commit that referenced this pull request Apr 19, 2024
* Adds text to lootpanel items [no gbp] (#82722)

## About The Pull Request
Just some qol, this captures a small portion of the item name and fixes
some icon clipping issues

I clipped it at 5 because monke
![Screenshot 2024-04-17
034211](https://github.com/tgstation/tgstation/assets/42397676/64d9ce18-3722-4382-a58d-f58eb6b9f26c)


![gXR2XjslzR](https://github.com/tgstation/tgstation/assets/42397676/3fc93110-e35e-434a-b63b-89aa669090e7)
## Why It's Good For The Game
Handy if youre in a rush and won't upgrade byond to fix the issue
## Changelog
:cl:
fix: Lootpanel additions: Condensed item names for the quick of draw
/:cl:

* Adds text to lootpanel items [no gbp]

---------

Co-authored-by: NovaBot <[email protected]>
Co-authored-by: Jeremiah <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants