Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-3220

Gray out right-click menu options when file does not have own/write permissions

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Situation: When using the right-click menu option in Community or Shared folders the Delete and Cut menu options are available even if the user does not have own/write permissions.

      Task: Gray out the Delete and Cut right-menu options when the user does not have own/write permissions for files and folders in the Community or Shared folders. Note that this behavior seems to have already been implemented for Rename.

        Attachments

          Activity

          Show
          karthik Karthik Raveendran added a comment - PR Submitted. https://bitbucket.org/nfreese/bioviz-connect/pull-requests/283/igbf-3220-gray-out-right-click-menu
          Hide
          nfreese Nowlan Freese added a comment -

          Merged

          Show
          nfreese Nowlan Freese added a comment - Merged
          Hide
          ann.loraine Ann Loraine added a comment -

          New code is deployed. https://bvctest3.bioviz.org is up to date.

          Show
          ann.loraine Ann Loraine added a comment - New code is deployed. https://bvctest3.bioviz.org is up to date.
          Hide
          nfreese Nowlan Freese added a comment -

          The cut/delete options are present in home folder, I go to community and they are grayed out, i go to shared and they are grayed out, and then if I go back to Home they are still grayed out (they should not be grayed out in Home).

          If I refresh the page they come back as an option.

          Rename seems to work correctly.

          Show
          nfreese Nowlan Freese added a comment - The cut/delete options are present in home folder, I go to community and they are grayed out, i go to shared and they are grayed out, and then if I go back to Home they are still grayed out (they should not be grayed out in Home). If I refresh the page they come back as an option. Rename seems to work correctly.
          Show
          karthik Karthik Raveendran added a comment - Changes made. PR submitted https://bitbucket.org/nfreese/bioviz-connect/pull-requests/284/igbf-3220-reset-pop-up-menu-options-for
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          I logged into bvctest3 and right-clicked on: gt30w140n90_oregon.tif, located in https://bvctest3.bioviz.org/base/#/iplant/home/shared/aegis/Spatial-bootcamp/basics/raster (a Community folder).

          All options were grayed out, indicating my user lacks permission to execute those functions as I am not the owner of that file.
          The only option that seems like it ought to also be grayed out but wasn't was "Manage Link," which may be the expected behavior at this time due to the fact that the "manage link" CyVerse endpoint is not properly working right now?

          Show
          ann.loraine Ann Loraine added a comment - - edited I logged into bvctest3 and right-clicked on: gt30w140n90_oregon.tif, located in https://bvctest3.bioviz.org/base/#/iplant/home/shared/aegis/Spatial-bootcamp/basics/raster (a Community folder). All options were grayed out, indicating my user lacks permission to execute those functions as I am not the owner of that file. The only option that seems like it ought to also be grayed out but wasn't was "Manage Link," which may be the expected behavior at this time due to the fact that the "manage link" CyVerse endpoint is not properly working right now?
          Hide
          nfreese Nowlan Freese added a comment -

          Merged

          Show
          nfreese Nowlan Freese added a comment - Merged
          Hide
          karthik Karthik Raveendran added a comment -

          For the above comment by Dr. [~aloraine], all though the user cannot change anything in the Manage Link, the user may want to see what the status of the link is i.e. whether it is public or not. Which is why, I think, we do not gray out the option in the pop-up menu but the create public link button is disabled so the user cannot change anything.

          Show
          karthik Karthik Raveendran added a comment - For the above comment by Dr. [~aloraine] , all though the user cannot change anything in the Manage Link, the user may want to see what the status of the link is i.e. whether it is public or not. Which is why, I think, we do not gray out the option in the pop-up menu but the create public link button is disabled so the user cannot change anything.
          Hide
          ann.loraine Ann Loraine added a comment -

          Ok - that sounds good to me!

          Show
          ann.loraine Ann Loraine added a comment - Ok - that sounds good to me!
          Hide
          ann.loraine Ann Loraine added a comment -

          New code deployed to https://bvctest3.bioviz.org.

          Show
          ann.loraine Ann Loraine added a comment - New code deployed to https://bvctest3.bioviz.org .
          Hide
          nfreese Nowlan Freese added a comment -

          Working correctly.

          Closing ticket.

          Show
          nfreese Nowlan Freese added a comment - Working correctly. Closing ticket.

            People

            • Assignee:
              karthik Karthik Raveendran
              Reporter:
              nfreese Nowlan Freese
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: