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

Separates cross-platform managers into separate table #2313

Closed
wants to merge 1 commit into from

Conversation

cskeeters
Copy link

As suggested in #1971.

@casey
Copy link
Owner

casey commented Aug 20, 2024

Looking at this, I think maybe it's better if we just put all the entries for Various at the top of the table, instead of splitting it into two , and maybe instead of Various call it Cross-platform. What do you think?

@cskeeters
Copy link
Author

That's what I did.

@casey
Copy link
Owner

casey commented Aug 21, 2024

My suggestion is to keep a single table, but to put the cross-platform options at the top of that single table.

@cskeeters
Copy link
Author

Oh I see. Personally, I like the sparate table. It's like two Hashmaps that have different keys. It tells the user you have to check both maps; one for the package manager and one for the OS. If you got rid of the OS column and ordered everything by package manager, that would be one thing, but then you'd loose some of the information you have about OS support. That said, I think you should do what makes sense to you.

@casey
Copy link
Owner

casey commented Aug 22, 2024

Just based on personal preference, I think I like a single unified table best, mostly because I don't like too deeply nested headings.

@casey
Copy link
Owner

casey commented Sep 21, 2024

I wound up doing this in #2385, and I think you're right, multiples tables are probably best.

@casey casey closed this Sep 21, 2024
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.

2 participants