Skip to content
This repository was archived by the owner on May 5, 2025. It is now read-only.

Conversation

@RoinujNosde
Copy link
Collaborator

Added API /clans/{tag}
Extracted addColors to Utils.php
Added ext-json as dependency
Added bootstrap.js
Added style.css with colors from 0 to 9
Added route view for details (modal)

Added API /clans/{tag}
Extracted addColors to Utils.php
Added ext-json as dependency
Added bootstrap.js
Added style.css with colors from 0 to 9
Added route view for details (modal)
@RoinujNosde RoinujNosde requested a review from Tomut0 April 7, 2021 01:22
2. Add colors.scss and settings for its compiling
* style.css | auto-compiled version for production
@Tomut0 Tomut0 linked an issue Apr 7, 2021 that may be closed by this pull request
@RoinujNosde RoinujNosde requested a review from Tomut0 April 7, 2021 17:56
let nick = element.dataset.nick;
if (tag) {
element.onclick = () => showClan(tag)
} else if (nick) {
Copy link
Owner

Choose a reason for hiding this comment

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

Why else if here? 👀

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

So it won't add a listener that does nothing except print "invalid id" on log.

@Tomut0 Tomut0 added the enhancement New feature or request label Apr 8, 2021
@Tomut0 Tomut0 assigned Tomut0 and RoinujNosde and unassigned RoinujNosde and Tomut0 Apr 8, 2021
@Tomut0 Tomut0 merged commit 3c977fe into Tomut0:dev Apr 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tags are not getting colored

2 participants