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

New Support Lists #135

Merged
merged 42 commits into from
Feb 21, 2017
Merged

New Support Lists #135

merged 42 commits into from
Feb 21, 2017

Conversation

DanielOaks
Copy link
Member

@DanielOaks DanielOaks commented Jan 13, 2017

Recently, we unbundled specs from explicit 3.1/3.2/etc releases on the specs page. However, our support lists still show everything as being under 3.1, 3.2, etc.

These new support lists are more extensible and combine the specs in a nice, easy-to-understand way that works better for how we're presenting and releasing specifications these days.

Examples

Clients page:
IRCv3 Clients page

Servers page:
IRCv3 Servers page


Thoughts?

@DarthGandalf
Copy link
Member

Generally, looks good!
Let's not hide the all-no rows. E.g. I don't see the roadmap what ZNC doesn't implement yet.
Ideally, the table should be split dynamically, depending on window size. There's lots of free space to both sides of screen.

@DanielOaks
Copy link
Member Author

Yeah dynamically splitting it requires javascript and all that fun stuff. If someone wants to take a shot, they're free to go ahead, but ye.

I agree with the all-no rows -- I think that was just a side-effect of my fun with the support list file. Will fix

@DanielOaks
Copy link
Member Author

Actually, I've made it the original width again. Here are some new screenies, looks a decent bit better:

Client page:
IRCv3 Client page

Server page:
IRCv3 Server page

@DanielOaks
Copy link
Member Author

Hmm, this is an example of what it looks like when you hover over an item:

new-support-list-hover

Solving that is difficult, because we need to convey a whole lot of info at once. I'll have a play, maybe manually marking tables to be 100% when they exceed a certain point or something similar.

@Zarthus
Copy link
Contributor

Zarthus commented Jan 14, 2017

(probably for a different issue)
I also think now would be a good time to ditch some of the inconsistent numbers like with ZNC having 0.094 ...

I would go with "yes" for things that aren't in git, and assume the user always uses the latest stable software or oldstable. It's not ircv3's concern to keep a changelog of each client.

@DanielOaks
Copy link
Member Author

Made the cols wider because it was difficult to read them (thanks @Zarthus for pointing it out!), so should be a fair bit nicer now. In addition, it now takes up the full width if it should (if the table has like 8 s/w columns or more).

Client page:
IRCv3 Clients page

@DanielOaks
Copy link
Member Author

Cool, I'll merge this tomorrow if there aren't any further issues with it

@jwheare
Copy link
Member

jwheare commented Jan 16, 2017

Can you do anything about the text shifting by a pixel when you hover? I think the tooltip whitespace is interfering.

@lrstanley
Copy link
Contributor

if this does get implemented, may be worth while to implement pull/139 into this as well, which adds support for showing server/library languages being used.

@DanielOaks
Copy link
Member Author

DanielOaks commented Jan 22, 2017

This isn't being merged right now. @attilamolnar has brought up concerns with the way that this displays specs wrt the unbundling and all. I'm giving peeps time to put forth some alternate suggestions on this page and the specs page -- since the display here matches the specs page, if there's something wrong here we need to fix it on the specs page first. As well, the specs page is vastly more easy to modify than this page, so changes should be leveled at that first.

@jwheare
Copy link
Member

jwheare commented Jan 22, 2017

Cheers Dan, shouldn't be hard to sort out. I'll have some time to chat after Monday.

@dequis
Copy link
Contributor

dequis commented Jan 23, 2017

FWIW I just noticed that the current version in master doesn't have any caps for chatzilla, but this PR fixes it in the sw_clients.yml part of b005cfd. May be worth backporting that to master if this PR is on hold.

Nevermind: #140

@DanielOaks
Copy link
Member Author

I'll resolve conflicts after things are worked out here

@DanielOaks
Copy link
Member Author

Aight so this should be fine to merge now. If there aren't any further concerns I'll merge this in the next few days.

@jwheare
Copy link
Member

jwheare commented Feb 16, 2017

Pushed some commits to fix labeled-response, updated the message-tags-0.2 version, and updated the IRCCloud level of support.

Live preview version here:
http://jwheare.github.io/ircv3.github.io/support/networks.html
http://jwheare.github.io/ircv3.github.io/software/clients.html
http://jwheare.github.io/ircv3.github.io/software/servers.html
http://jwheare.github.io/ircv3.github.io/software/libraries.html

@DanielOaks DanielOaks merged commit 2b64b85 into master Feb 21, 2017
@DanielOaks DanielOaks deleted the master+new-support-lists branch February 21, 2017 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants