Conversation
9818e0b to
9e8921b
Compare
hjpalpha
left a comment
There was a problem hiding this comment.
imo using dynamic tabs to shorten the page/scrolling might be something to consider
and we should try to fetch the current matchgroup too (and once we have standardized standings and can build standings from lpdb stabdings data also display the group if there is one attached)
right now it is using whatever is available at match opponent level |
|
fwiw the "This team has not played any matches yet." seems suboptimal in cases where opponents have no h2h maybe we should make that something like "Liquipedia has no recorded Head to Head for these opponents" |
i think we will just leave that as is^^ |
|
fyi, the /Team version in dev doesn't match what is in the PR (the stuff in the PR throws currently on cs due to a missing nil check) |
|
i was playing around a bit with this:
https://liquipedia.net/starcraft2/User:Hjpalpha/wip17 among other things i was trying to push the bottom content (h2h) into a tab too (had to jump into meetings when writing this) |
|
|
fwiw i personally think it would be nice to show it, but imo not a blocker |
Co-authored-by: hjpalpha <75081997+hjpalpha@users.noreply.github.com>
|
needs rebase |
|
@ElectricalBoy would you be opposed to always using tabs (like on sc2) if there is no matchpage setup? |
I don't mind |
| if Array.any(match.opponents, Opponent.isBye) then | ||
| return |
There was a problem hiding this comment.
Why would there be matches with BYEs that have streams set, exact dates, and aren't finished 🤔
There was a problem hiding this comment.
if an opponent drops out and the other opponent isn't determined yet and someone forgot to remove the date or it is auto inherited
There was a problem hiding this comment.
If it is inherited, it is not set to be exact, right?
| MatchPage(self.matches[1]):render(), | ||
| HtmlWidgets.H3{children = 'Channel Schedule'}, | ||
| self:_createMatchTicker():query():create() |
There was a problem hiding this comment.
Would it make sense to also use the grid here? the Full-width matchticker looks a bit weird.
Otherwise could use the entity variant of the ticker.
There was a problem hiding this comment.
since this is the variant that gets used if there is a match page i would say electrical boy is better suited to state the opinion of wikis that use such
Co-authored-by: SyntacticSalt <mail@mbergen.de>
Co-authored-by: SyntacticSalt <mail@mbergen.de>
Co-authored-by: SyntacticSalt <mail@mbergen.de>




Summary
Resolves #6652
How did you test this change?
StreamPage/Team, custom is oos for now)