-
Notifications
You must be signed in to change notification settings - Fork 0
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
More context in homepage #4
Comments
Could you please give it another look? I tried to add more context. And styled the page to my taste. Please feel free to change anything you'd like to change. |
Looks great! However, now the backends need to be hardcoded in the CSS, which is suboptimal. How does it do with backends that do not correspond to a CSS class? I was just thinking, when we split this to a template repo and a fork, we should have "Madata Authentication Provider" in the template repo and "Madata Official Authentication Provider" only in our fork. PS: We may not want to do a template repo, as it will be a pain to pull in changes, see https://stackoverflow.com/questions/56577184/github-pull-changes-from-a-template-repository |
What if we provide a placeholder icon based on the Madata logo (e.g., with some sort of a badge, like “backend”)? Later, devs can replace it with their own logo and/or redefine only one simple rule in their CSS to reflect the changes. What do you think? |
SGTM. Or maybe a stylized icon based on the first letter of the backend, like some avatars? |
Though I'm worried this is spending effort on things that are not on the critical path |
Are we done with that? |
The current index page on auth.madata.dev doesn't provide any context for someone visiting it:
It should have some sort of heading (e.g. "Madata Authentication Provider"), with a link to documentation, and it should list the backends instead of linking to a JSON file.
The text was updated successfully, but these errors were encountered: