azerothcore
Feature/my improvements
Summary: This Pull Request improves the UI/UX of the server-status project and enhances client-side functionality. No impact on server side.
Key Changes: [✅] Redesigned the UI with a modern, fantasy-themed style (SCSS refactor). [✅] Implemented dynamic column sorting with clear visual indicators. [✅] Improved search functionality (filtering by player name and guild). [✅] Optimized SCSS for better cross-browser compatibility (e.g., Firefox rendering fixes). [✅] Minor cleanup: aligned angular.json config with Angular documentation.
Benefits: [✅] Better user experience with cleaner visuals, World of Warcraft vibes. [✅] Fully client-side sorting & filtering to reduce server load.
Let me know if adjustments are needed! Happy to align with project standards. :)
Hi!
It seems very cool, but I would like to keep the old style under a conf parameter, you could use config.ts for it.
Hey, just to tell that I've been working hard (like hard) to get a theme system out. I am not a professional devops, doing this to support this amazing project :)
All done (except for the app.model.ts part). I decided not to use the config.ts approach for theme management, as it felt much cleaner and simpler to handle it directly through 'style.scss' by commenting or uncommenting the desired themes with '@use'. I'm quite happy with the result.
Thanks for pushing me to improve and aim for better practices!
Hi, I think that there should be an easy way to make the theme configurable, I will have a look to the changes and the conf, then I will merge the PR
Done, I made a conf for enabling/disabling by config.ts the azeroth-theme, let me know if it's ok for you, make some tests please @benoitheylens
thanks by the way!
Hey! For me, this PR brings a lot of cool features and having a second optional theme is great, I would see it even here https://github.com/azerothcore/arena-stats , indeed I kept the first theme here to keep the color compatibility with the arena-stats.
Anyways, if you think that this PR is good I would merge it, I tested it locally and it seems good for me.
You could reuse the CSS for arena-stats to keep things consistent. I had to work around some issues with Bootstrap to make sure corners look the same in Chrome and Firefox.
I used free/open-source fonts and tried to keep things simple.
Tested on mobile and tablet: looks good.
Ready to ship 🫡