-
Notifications
You must be signed in to change notification settings - Fork 5
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
added hyperlink to spaceall heatmap on status button #62
base: master
Are you sure you want to change the base?
Conversation
@@ -18,6 +18,7 @@ <h1>Entropia</h1> | |||
</div> | |||
|
|||
<div class="club-status-wrapper"> | |||
<a href="https://mapall.space/heatmap/show.php?id=Entropia" target="_blank" rel="noopener noreferrer"></a> | |||
<span class="club-status"></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich frag mal ganz naiv, sollte sich das </a>
nicht eher nach dem <span class="club-status"></span>
Schließen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Disclaimer, wenig Ahnung von HTML. Daher naiv zurückgefragt; wieso? Bzw kommt drauf an, so viel ich weiß?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Man möchte ja den Teil den man anklickt innerhalb des <a>
tags haben. Und das ist ja eigentlich diese Club Status anzeige...
Beispiel:
<a href="http://example.com">Dieser Text führt nach example.com</a>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vielleicht ergänzend ein bisschen Kontext:
<span class="club-status"></span>
ist das Element, in welches per Javascript nach Abfrage der SpaceAPI das Wort "geöffnet" bzw. "geschlossen" geschrieben wird und welches, ebenfalls per Javascript, eine zusätzliche Klasse erhält, um die grüne oder rote Hintergrundfarbe zu setzen.
Es könnte also auch das <span>
durch den vorgeschlagenen <a>
-Tag mit der Klasse club-status
ersetzt werden.
Besteht der Wunsch noch bzw. hat jemand die Vorschläge umgesetzt? |
Umgesetzt bisher nicht, aber würds gern noch machen. Ich setz mich später nochmal ran, sofern das nicht jemand anders schnell umschreibt ^^ |
i added a simple feature wished by some other people. when you click on the status banner, a new tab will be opened with the heatmap from spaceall. the hover for last change feature is still there.