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

PAV-415 Improve Magpie UI #2

Merged
merged 19 commits into from
Oct 4, 2017
Merged

PAV-415 Improve Magpie UI #2

merged 19 commits into from
Oct 4, 2017

Conversation

fmigneault
Copy link
Collaborator

@fmigneault fmigneault commented Sep 27, 2017

Changes

  • replaces text input by combobox for services' resources according to their respectively allowed resources (ie: only file/directory selectable within thredds, only workspace for WMS)
  • removes possibility to add children resources in treeview for file
  • replace radio buttons in pages add_user, add_service and login by combobox control
  • make input forms with comboboxes of same size as input text
  • change buttons side-by-side for view_users, view_groups and view_services
  • make red button for delete operations
  • make disabled grey button for not permitted operations according to context (add_child / delete)
  • dynamically change home page buttons position according to available screen width

Extra

  • made code cleanup according to multiple found deprecation warnings
  • fixed some make / compile issues
  • fixed error resulting from invalid ServiceWFS class inheritance

@fmigneault fmigneault self-assigned this Sep 27, 2017
@fmigneault fmigneault force-pushed the PAV415-Improv-UI branch 3 times, most recently from ab24711 to d6c2ea5 Compare October 3, 2017 17:23
@fmigneault fmigneault requested review from dbyrns and fderue October 3, 2017 19:04
@dbyrns
Copy link
Contributor

dbyrns commented Oct 4, 2017

:shipit:

@fmigneault fmigneault merged commit ce70c6b into master Oct 4, 2017
@fderue fderue deleted the PAV415-Improv-UI branch October 13, 2017 21:01
@fmigneault fmigneault mentioned this pull request Oct 26, 2020
15 tasks
@ChaamC ChaamC mentioned this pull request Jan 13, 2021
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.

3 participants