Skip to content

Conversation

@sanderDijkxhoorn
Copy link
Contributor

Implementing #1101

image
image
image

Latest design:

image

I made a little oopsy with the last PR... here is a cleaner version of the PR, no GitHub workflow commits.

@lenisko lenisko mentioned this pull request Jun 3, 2025
@Mygod
Copy link
Collaborator

Mygod commented Jun 3, 2025

By the way the threshold numbers are definitely not one/two thirds. I think it's 73% or something similar.

@Mygod
Copy link
Collaborator

Mygod commented Jun 3, 2025

Yeah it's ~47% and then ~73% as the threshold, or more precisely, 7/15 and then 11/15. The minimum is 20% or 1/5.

See also: https://pogo.gamepress.gg/gym-motivation-what-you-should-know

@lenisko
Copy link
Contributor

lenisko commented Jun 4, 2025

I made another small change (dropped defender info from extra text), including hearth thing @Mygod mentioned: sanderDijkxhoorn#3

feat(client): drop defenders from extra info, fix defenders hearth
@sanderDijkxhoorn
Copy link
Contributor Author

I merged it, could someone try to run this PR and check if all functions correctly?

@lenisko
Copy link
Contributor

lenisko commented Jun 4, 2025

@sanderDijkxhoorn I'm running it on prod at top of #1113. Everything looks good related to defenders.

@Mygod
Copy link
Collaborator

Mygod commented Jun 4, 2025

image
White stripes are not visible in light mode. 👀

@Mygod
Copy link
Collaborator

Mygod commented Jun 4, 2025

By the way, do we want to display some of these other fields? deployed_time,battles_won,battles_lost,times_fed

@lenisko
Copy link
Contributor

lenisko commented Jun 5, 2025

By the way, do we want to display some of these other fields? deployed_time,battles_won,battles_lost,times_fed

Waste of space... Unless we would toggle ON CLICK between Pokemon name & CP line and something like:

:mdiMedal: {battles_won} :mdiEmoticonSad: {battles_lost} :mdiFoodApple: {times_fed}
{deployed_time}

@Mygod
Copy link
Collaborator

Mygod commented Jul 7, 2025

I guess if people are okay with it, I will merge and implement the changes myself.

@Mygod
Copy link
Collaborator

Mygod commented Jul 7, 2025

image

Looks fucking good. UI done by vibe coding. @sanderDijkxhoorn @lenisko Merge?

@sanderDijkxhoorn
Copy link
Contributor Author

image

Looks fucking good. UI done by vibe coding. @sanderDijkxhoorn @lenisko Merge?

Those icons look a little odd, replace it with berry and such?

Why does the top one start with 1d xxxx
The others have some hour minute seconds?

@Mygod
Copy link
Collaborator

Mygod commented Jul 7, 2025

Minor complaints about UI will be only accepted in the form of commits or PRs. :)

@Mygod Mygod merged commit 3f5d40d into WatWowMap:develop Jul 7, 2025
2 checks passed
@github-actions
Copy link

github-actions bot commented Jul 7, 2025

🎉 This PR is included in version 1.36.0-develop.19 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@lenisko
Copy link
Contributor

lenisko commented Jul 7, 2025

@Mygod this looks like a 💩 We just literary been fighting for any vertical space, and you just added a bunch of useless stuff on the same page.

That's why I proposed additional data on toggle/click.

@Mygod
Copy link
Collaborator

Mygod commented Jul 7, 2025

@lenisko I just care about functionality. If you want to make it look less 💩, PR welcome. :)

@Mygod Mygod mentioned this pull request Jul 16, 2025
@github-actions
Copy link

🎉 This PR is included in version 1.36.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

4 participants