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

240 map list #242

Merged
merged 58 commits into from
Jun 2, 2024
Merged

240 map list #242

merged 58 commits into from
Jun 2, 2024

Conversation

snixtho
Copy link
Member

@snixtho snixtho commented Jun 1, 2024 •
edited
Loading

  • New default theme
  • Map List (Kinda basic for now)
  • Map Queue
  • Various fixes
  • Manialink-based UI Styles are now separated into their own styles

New UI Components

  • Backdrop
  • Chip
  • ConfirmDialog
  • LinkButton
  • Rating
  • ScrollBar
  • Separator
  • Tag
  • Circle
  • Rectangle

@snixtho
initial
efd6b7c
@snixtho
implement async deque
bb7d777
@snixtho
implement queue controller
a41f9f6
@snixtho
Add container and panel ML components.
c25dbcf
@snixtho
add border to panel
0c56550
@snixtho
convert opacity based on 4-byte color
b363a64
@snixtho
add chip
0df0e3b
@snixtho
separator
e799aa5
@snixtho
add placeholder for textinput
7d1d6e9
@snixtho
smol save
0ca5557
@snixtho
adfg
e6d67d7
@snixtho
map list module project file
7c6fde9
@snixtho
local save
541607c
@snixtho
changes
0be57df
@snixtho
map row tweaks
22b665e
@snixtho
basic scrollbar
ff584be
@snixtho
scrollbar can now control the scrolling element
67c5bea
@snixtho
fix relative drag position of scrollbar
2d4302e
@snixtho
begin using map data objects
d2735df
@snixtho
actually show the current map list
d6a8218
@snixtho
save map details to db
91ab4f0
@snixtho
Optimize GetCurrentMapListAsync
5f99ddc
@snixtho
Add ability to queue maps from the maplist.
82e6971
@snixtho
add widget prototype with new style
10a0cda
@snixtho
starting new style
13ee3f2
@snixtho
new button and dropdown
ec217ba
@snixtho
text input
ad22b40
@snixtho
huh
03e85c8
@snixtho
alert
ec85050
@snixtho
updated checkbox
af259f7
araszka
araszka approved these changes Jun 1, 2024
View reviewed changes
src/EvoSC.Manialinks/Templates/Controls/Text.mt Outdated Show resolved Hide resolved
src/EvoSC.Manialinks/Templates/Controls/TextInput.mt Outdated Show resolved Hide resolved
src/EvoSC.Manialinks/Templates/Drawing/Circle.mt Outdated Show resolved Hide resolved
src/EvoSC.Manialinks/Templates/Scripts/Chip.ms Outdated Show resolved Hide resolved
AtomicLiquid
AtomicLiquid approved these changes Jun 2, 2024
View reviewed changes
Copy link
Member

@AtomicLiquid AtomicLiquid left a comment

Choose a reason for hiding this comment

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

Havent been able to review the tests, but rest looks good, just a few minor things.

src/EvoSC.Common/Interfaces/IServerClient.cs Show resolved Hide resolved
src/EvoSC.Common/Interfaces/Services/IMapService.cs Show resolved Hide resolved
src/EvoSC.Common/Interfaces/Services/IMapService.cs Show resolved Hide resolved
src/EvoSC.Manialinks/Templates/Containers/Container.mt Outdated Show resolved Hide resolved
src/EvoSC.Manialinks/Templates/Scripts/Wigets.ms Outdated Show resolved Hide resolved
src/Modules/ExampleModule/Templates/DialogExample.mt Show resolved Hide resolved
src/Modules/MapListModule/Interfaces/IMapListService.cs Outdated Show resolved Hide resolved
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Jun 2, 2024

Quality Gate Passed Quality Gate passed

Issues
13 New issues
0 Accepted issues

Measures
0 Security Hotspots
24.9% Coverage on New Code
2.3% Duplication on New Code

See analysis details on SonarCloud

@snixtho snixtho merged commit 91610d0 into master Jun 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@araszka araszka araszka approved these changes

@AtomicLiquid AtomicLiquid AtomicLiquid approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@snixtho @araszka @AtomicLiquid