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

FormSpec: Support custom size and spacing for slots in list (#7971) rebased #9240

Closed
wants to merge 2 commits into from

Conversation

Unarelith
Copy link
Contributor

@Unarelith Unarelith commented Dec 24, 2019

See #7971, I just rebased it in another branch to make things easier for me.

There is a small problem with this one after rebase though, when resizing craft preview over it's original size, the square will stay the same size but the item inside will scale up. I don't understand why it's happing because it's the only slot with an issue, other slots are fine.

I tried to make the exact same changes in the files and I don't remember much about how this PR works, especially because I wasn't the original author.

screenshot-20191224141345

@Unarelith
Copy link
Contributor Author

Ok it's fixed now. Ready for review.

@SmallJoker SmallJoker added this to PRs: Highest priority on top in Formspec Priority List Dec 30, 2019
@rubenwardy rubenwardy added the Rebase needed The PR needs to be rebased by its author. label Mar 1, 2020
@rubenwardy
Copy link
Member

This will need to be reworked to support styling, and also use the new class for inventory lists

@Unarelith
Copy link
Contributor Author

Sorry, I'm dropping this PR, I don't have time for this because of OpenMiner.

@Unarelith Unarelith closed this Mar 2, 2020
@v-rob v-rob moved this from PRs: Highest priority on top to Adoption needed or rejected in Formspec Priority List Aug 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature ✨ PRs that add or enhance a feature Formspec Rebase needed The PR needs to be rebased by its author.
Projects
Formspec Priority List
  
Adoption needed or rejected
Development

Successfully merging this pull request may close these issues.

None yet

3 participants