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.

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

eureka: Add a BA Portal Timer to Hydatos #3764

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

KattTails
Copy link
Contributor

@KattTails KattTails commented Dec 8, 2021

This adds a new timer to the Hydatos map for tracking the blue/red BA portals in the same style as the weather info

Preview image:
image
image

I've also implemented a new check for nm.startTimerOnKill, which allows NMs/CEs such as Ovni, Castrum and the Dalriada to be tracked more accurately as a result.

If you want to test it quickly you can use this url: https://katttails.github.io/cactbot/ui/eureka/eureka.html
however do note that I've been doing some other work on top of this PR so there are some other unrelated code changes as well, but they shouldn't affect anything (I hope).

Needs translations.

@cactbotbot
Copy link
Collaborator

cactbotbot commented Dec 8, 2021

@KattTails Thanks for your contribution! 🌵🚀

@KattTails KattTails changed the title Eureka: Add a BA Portal Timer to Hydatos eureka: Add a BA Portal Timer to Hydatos Dec 8, 2021
@KattTails
Copy link
Contributor Author

cactbot bot doesn't like capitals :(

ui/eureka/eureka_config.ts Outdated Show resolved Hide resolved
@@ -251,6 +252,15 @@
border-bottom: 2px dashed rgba(255, 255, 255, 0.3);
}

#portal-timer-container {
Copy link
Owner

Choose a reason for hiding this comment

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

What do you think about combining these with weather (like on top of the current weather?) just to avoid having too many boxes spread about?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I decided on creating a separate box for it because the weather container is already pretty tall compared to the Hydatos map and I wanted to keep it as compact as possible.
The layout of the weather box is also pretty fixed, there's always the time of day and then 5 weathers visible, whereas the portal timer is only going to be popping up when it's relevant to be shown. I think that having the box resize and an extra element appearing/disappearing would be pretty distracting personally, but I'm open to others' thoughts on this.

Copy link
Owner

Choose a reason for hiding this comment

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

Ok, that seems reasonable!

@github-actions github-actions bot removed the config label Dec 12, 2021
@@ -1,4 +1,4 @@
<html lang="en-US">
<html>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<html>
<html lang="en-US">

@@ -11,6 +11,18 @@ export const bunnyLabel: LocaleText = {
ko: '토끼',
};

const sec2time = (timeInMs: number) => {
Copy link
Owner

Choose a reason for hiding this comment

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

is this ms2time and not sec2time? or, maybe I misunderstand what "sec" means here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants