Skip to content

refactor: major code restructuring#415

Merged
ErikKalkoken merged 69 commits intomainfrom
refactor-info-window
Mar 8, 2026
Merged

refactor: major code restructuring#415
ErikKalkoken merged 69 commits intomainfrom
refactor-info-window

Conversation

@ErikKalkoken
Copy link
Owner

@ErikKalkoken ErikKalkoken commented Mar 5, 2026

The ui package has grown very large over time. This PR breaks it down into multiple smaller packages.

The (old) ui package has been split by feature into the following:

  • assetui: asset widgets
  • characterui: character widgets
  • characterwindow: manage character window
  • corporationui: corporation widgets
  • gamesearchui: game search widgets / window
  • industryui: industry widgets
  • settingswindow: settings window
  • skillui: skill widgets
  • statuswindow: update status window
  • ui (new): baseUI, desktopUI, mobileUI and desktop related widgets

@codecov
Copy link

codecov bot commented Mar 5, 2026

Codecov Report

❌ Patch coverage is 6.53009% with 3650 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.7%. Comparing base (8287bc8) to head (73ae6a7).
⚠️ Report is 70 commits behind head on main.

Files with missing lines Patch % Lines
internal/app/infowindow/infowindow.go 0.0% 486 Missing ⚠️
internal/app/characterwindow/charactertags.go 0.0% 385 Missing ⚠️
internal/app/characterwindow/characteradmin.go 0.0% 229 Missing ⚠️
internal/app/infowindow/characterinfo.go 0.0% 215 Missing ⚠️
internal/app/characterservice/section.go 14.0% 177 Missing and 1 partial ⚠️
internal/app/infowindow/corporationinfo.go 0.0% 153 Missing ⚠️
internal/app/characterui/contracts.go 0.0% 105 Missing ⚠️
internal/app/assetui/assetsearch.go 0.0% 96 Missing ⚠️
internal/app/characterwindow/charactertraining.go 0.0% 95 Missing ⚠️
internal/app/corporationservice/section.go 10.4% 94 Missing ⚠️
... and 72 more
Additional details and impacted files
@@           Coverage Diff           @@
##            main    #415     +/-   ##
=======================================
- Coverage   34.3%   30.7%   -3.6%     
=======================================
  Files        254     273     +19     
  Lines      41077   41378    +301     
=======================================
- Hits       14102   12743   -1359     
- Misses     24798   26788   +1990     
+ Partials    2177    1847    -330     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ErikKalkoken ErikKalkoken force-pushed the refactor-info-window branch from bc8fe3a to 53fdfd5 Compare March 7, 2026 16:06
@ErikKalkoken ErikKalkoken force-pushed the refactor-info-window branch from 53fdfd5 to d0774ba Compare March 7, 2026 16:08
@ErikKalkoken ErikKalkoken changed the title refactor: info window refactor: major restructuring Mar 7, 2026
@ErikKalkoken ErikKalkoken changed the title refactor: major restructuring refactor: major code restructuring Mar 7, 2026
@ErikKalkoken ErikKalkoken merged commit 5786c20 into main Mar 8, 2026
10 checks passed
@ErikKalkoken ErikKalkoken deleted the refactor-info-window branch March 8, 2026 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant