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.

Sign up for GitHub

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

Jump to bottom

0.2.2 (Color Picker Update) #308

Merged
merged 20 commits into from
Dec 24, 2023
Merged

0.2.2 (Color Picker Update) #308

merged 20 commits into from
Dec 24, 2023

Conversation

Wyvest
Copy link
Member

@Wyvest Wyvest commented Dec 19, 2023 •
edited
Loading

Description

Changelog:

  • Add new TinyFD API
    • TinyFD is a cross-platform file selection dialog API
    • On Linux, TinyFD "allows shell metacharacters in titles, messages, and other input data," meaning it is vulnerable to command injection. Hence, all user input should be treated as untrusted, and should be sanitized before passing to TinyFD.
  • Add new SkyClient-exclusive launch message
  • Fix dependency tooltips requiring a description to render
  • Update Kotlin to 1.9.10 (and accompanying libraries)
  • Fix fast render / anti-aliasing support with OneConfig
  • Allow changing of bracket color in text HUDs
  • Fix changing to HUD programmatically not applying visually
  • Fix NPEs in GuiUtils when opening a null (empty) screen
  • NEW COLOR PICKER GUI
    • Fixes any pre-existing bugs with the old color selector
  • Remove Quiltflower from credits GUI
  • Fix seconds not being converted to milliseconds in OneColor constructor
  • Allow NanoVG flags to be modified in setupAndDraw method
  • Add methods for drawing ellipses via NanoVG
  • Add methods for drawing centered text via NanoVG
  • Fix glitches with text inputs
  • Fix NPE when interacting with / closing blank config GUIs
  • Add ability to set the alignment of HUDs
  • Make bordered text rendering thinner

Related Issue(s)

Checklist

  • I made a clear description of what was changed
  • I updated documentation or said what needs to be updated

Wyvest and others added 19 commits December 3, 2023 15:05
@Wyvest @nextdayy @xtrm-en
Color Picker V2 (but with NanoVGHelper) (#175)
868fa00 
* color picker pr but with nanovghelper

Co-Authored-By: nextdaydelivery <[email protected]>

* me after i run `apiDump` for the 49th time

* why is this here

* i found out why 💀

* fix

* apiDump

* update copyright year

* fix: fix input scissor not resetting

* fix: fix more input bugs

* 🐛 fix: fix random error

* chroma selector

* ignore block

* apiDump

* fix not translating properly and dropdown input

* fix merge conflicts

* A lot of QC changes

* "Refactor mouse input and resolution in ColorSelector"

Two additional imports were added to the ColorSelector.java, UMouse and UResolution from the universal library. The change was made to improve the calculation of the color-picking function. Previously, the mouse's X and Y coordinates were obtained from platform-specific code, and now the coordinates are read directly from the UMouse class. Also, the viewport height is now retrieved from UResolution, making the code more streamlined and cross-platform.

* more QC changes

---------

Co-authored-by: nextdaydelivery <[email protected]>
Co-authored-by: xtrm <[email protected]>
Co-authored-by: nxtdaydelivery <[email protected]>
@RedthMC
fix speed slider vibing (#300)
e2d82e1 
* fix speed slider vibing

* added checkerboard background
@MicrocontrollersDev
remove quiltflower (#301)
7978024
@ImToggle
better dropdown and color slider (#302)
ee77263
@realRobotix
Fix OneColor Chroma constructor (#304)
784fd22
@realRobotix
Make nvg flags accessible (#305)
e5a2794 
* Make nvg flags accessible

* fix doc formatting

* rebuild api
@realRobotix
add methods for drawing ellipses (#306)
afcae41
@ImToggle
config input fix (#307)
3ee2427 
fix
@Wyvest
make sure blank descriptions don't render
ae6f883
@Wyvest
Fix NPE when interacting / closing blank config GUI
1200fad
@Wyvest
Fix textbox inputs being broken in chroma
18fdcef
@Wyvest
Add SkyBlock toggle for HUDs
ccf06e1
@Wyvest
Add manual alignment of HUDs
e8a0722
@Wyvest
Run apiDump
1dbaa32
@Wyvest
Merge branch 'master' into develop
e2250d7
@Wyvest
make bordered text less thick
035e977
@Wyvest
change name from alignment to positionAlignment
0e5f7ab
@Wyvest
New: drawCenteredText in NanoVGHelper (#279)
45faf24 
* draw centered text

* icon

* Delete icon.svg

* Add files via upload

* Delete icon.svg

* run apiDump

* remove nvgFill call

---------

Co-authored-by: Wyvest <[email protected]>
@Wyvest
v0.2.2
a1843c4
@Wyvest Wyvest marked this pull request as ready for review December 22, 2023 00:18
pauliesnug
pauliesnug previously approved these changes Dec 22, 2023
View reviewed changes
Copy link
Member

@pauliesnug pauliesnug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me (check internal)

xtrm-en
xtrm-en previously approved these changes Dec 22, 2023
View reviewed changes
nextdayy
nextdayy previously approved these changes Dec 22, 2023
View reviewed changes
Copy link
Member

@nextdayy nextdayy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

DeDiamondPro
DeDiamondPro approved these changes Dec 22, 2023
View reviewed changes
Copy link
Member

@DeDiamondPro DeDiamondPro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one small note

@Wyvest Wyvest dismissed stale reviews from nextdayy, xtrm-en, and pauliesnug via 5a87b38 December 23, 2023 15:48
@Wyvest Wyvest merged commit 8046e5c into master Dec 24, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DeDiamondPro DeDiamondPro DeDiamondPro approved these changes

@xtrm-en xtrm-en xtrm-en left review comments

@pauliesnug pauliesnug pauliesnug left review comments

@nextdayy nextdayy nextdayy left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@Wyvest @xtrm-en @pauliesnug @DeDiamondPro @nextdayy @RedthMC @MicrocontrollersDev @ImToggle @realRobotix