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

Fix visuals in privacy share permissions view #2492

Merged
merged 3 commits into from
Feb 25, 2015

Conversation

viddo
Copy link
Contributor

@viddo viddo commented Feb 25, 2015

Fixes #2363

  • Moved back button to header
  • Removed subheader
  • Updated affected views due to class name changes

privacy-visual-changes

- Moved back button to header
- Removed subheader
- Updated affected views
@viddo
Copy link
Contributor Author

viddo commented Feb 25, 2015

@saleiva @xavijam review, please

@viddo
Copy link
Contributor Author

viddo commented Feb 25, 2015

by request, @javierarce can you review this too?

@saleiva
Copy link
Contributor

saleiva commented Feb 25, 2015

👍

@javierarce
Copy link
Contributor

👀 Looks fine to me 👍

Hide eventual overflow on x-axis since it should not be needed in the
first place. Show y-axis scroll only if necessary
@Cartofante
Copy link
Collaborator

Frontend tests were OK 👍 (details)

viddo added a commit that referenced this pull request Feb 25, 2015
Fix visuals in privacy share permissions view
@viddo viddo merged commit 6466059 into master Feb 25, 2015
@viddo viddo deleted the 2363-fix-share-permissions-header branch February 25, 2015 12:35
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.

Fixes on Share with your colleagues modal
5 participants