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

Feature simplify ui #6

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

saxtouri
Copy link

@saxtouri saxtouri commented Feb 22, 2017

This patch provides a simpler UI:

  • clean-cut bootstrap-based look and feel
  • InAcademia logo on top
  • responsive
  • improved English phrases
  • Obligatory (locked) fields are not shown by default (but users have the option to see them if they want)

Screenshots for desktop and mobile:
full-width
mobile

@leifj
Copy link

leifj commented Feb 24, 2017

this is not the main satosa repo - its at https://github.com/SUNET/SATOSA

@leifj
Copy link

leifj commented Feb 24, 2017

Also - lets go ahead and resolve IdentityPython/SATOSA#61 and get a PR against that repo

@saxtouri
Copy link
Author

@leifj This is a contribution to the consent module, not SAtoSA, that's why the request is here.

@leifj
Copy link

leifj commented Feb 24, 2017

Aha I see. The message to satosa-dev confused me.

@surfnet-niels
Copy link

I note the logo is now hard-coded. Could the location be moved to the CM model config, of better perhaps be injected as part of the CSS?

@surfnet-niels
Copy link

What happens when I select "Click to see what else..."? Could you provide a screen-shot of that as well?

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

3 participants