-
-
Notifications
You must be signed in to change notification settings - Fork 72
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
Anst/improvements #338
Anst/improvements #338
Conversation
I just noticed that there are no docker images for Unity alpha versions. Thus, the GitHub action cannot be upgraded to the newest version... |
@basisbit , I want to merge this stuff monday to have a more polished version for the Kickstarter. |
…ently asked questions - therefor, added an AccordionItemControl that shows/hides content chapters
… with icon PlayerProfileOptions: delete button red with icon
- therefor, collect song issues in SongMetaManager
…30 seconds or half of the song's duration
…rn songIssues alongside songMetas
…ight. - for more players, the notes are shown in the TimeBar independently of the specific player
- ScrollingNoteStreamDisplayer shows no dedicated lyrics bar. Instead, lyrics on notes can be used.
- use new placeholder property for search TextField
- dialog is shown until end of 2022 only once per start
- MaterialIcon and BootstrapIcon both extend new common base class FontIcon - load codepoints of font icons from TextAsset
…er song folder - created own class for SongFolderListEntryControl
- increased LeanTween max anim count
…me and tag value. - missing tag value is handled like empty string or 0 - space around colon is trimmed - added tests for this
no implicit width, margin, and padding on a button
…oes not update on reset
71dd975
to
ba2ad0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a lot. thanks for your continuous work!
What does this PR do?
A bunch of fixes and improvements.
Upgrade Unity
Song issue dialog
Song parsing
empty string
or0
value.Song folder checks
../
in the path or a trailing slashSimpler download scene UI
UltraStar song info
Font Icons
FontIcon
(base class)MaterialIcon
andBootstrapIcon
have been added.icon
property.Layout and styles
--control-width
asmin-width
now. This has the advantage that they can grow large if needed (depending on translated text)New logo
Particle effect when hitting golden notes
Lyrics are not cropped anymore
overflow: visible;
on notes to fix Lyrics are cropped on Android (Google Pixel 6) #324Show static lyrics with scrolling note stream
"bad" sentence rating is not shown anymore
Song started statistic
Show up to 3 player notes in time bar
Kickstarter info dialog
Closes Issue(s)
close #337
close #324
Additional Notes
Sorry for the hugeness of this PR. I just wanted to fix all this stuff before the upcoming Kickstarter.