Skip to content

UI problem in sharing options #5916

Closed

Description

@GitHubUser4234

Hi,

In NC12, after a user has unchecked "Can create" checkbox in the sharing options of a folder, it looks like he cannot uncheck the "Can change" checkbox subsequently. The whole page has to be refreshed for him to be able to uncheck the "Can change" checkbox also. If the page is not refreshed, the only thing that visibly happens is that all checkboxes are disabled briefly and then enabled again.

Activity

  1. MariusBluem commented on Jul 29, 2017

    @MariusBluem
    Member

    Could you please follow our issue template in the future? This helps us to reproduce issues more effective and faster ;) Please provide us some details about your environment (browser, ...), thanks!

  2. GitHubUser4234 commented on Jul 31, 2017

    @GitHubUser4234
    ContributorAuthor

    @MariusBluem Here is a gif where the issue can be quickly seen (gif might take a while to load). For each successful unchecking of a checkbox, the whole page needs to be refreshed (there are 2 refreshes in this gif):

    ice_video_20170731-150249 1

    Browsers used: Chrome 60.0.3112.78 and Firefox 54.0.1
    OS: Windows 10 Pro 64bit

  3. guiweber commented on Aug 21, 2017

    @guiweber

    I am also seeing this on my setup. Let me know if more information would help.

    Server configuration

    Operating system: Fedora 26
    **Web server: Apache 2.4.27
    **Database: MariaDB
    **PHP version: 7.1.8
    Nextcloud version: 12.0.2
    Updated from an older Nextcloud/ownCloud or fresh install: Updated from 12.0.0
    Where did you install Nextcloud from: Web Installer

    Client configuration

    Browser: Firefox 55.0.2
    Operating system: Windows 10

  4. MorrisJobke commented on Sep 3, 2017

    @MorrisJobke
    Member

    Maybe somebody from @nextcloud/javascript want to look into this

  5. added
    good first issueSmall tasks with clear documentation about how and in which place you need to fix things in.
    on Sep 3, 2017
  6. skjnldsv commented on Sep 4, 2017

    @skjnldsv
    Member

    Can't reproduce on 13.
    The can edit check is behaving as required.

  7. MorrisJobke commented on Sep 4, 2017

    @MorrisJobke
    Member

    @skjnldsv Could you test 12 as well please? :)

  8. self-assigned this
    on Nov 5, 2017
  9. added this to the Nextcloud 12.0.4 milestone on Nov 5, 2017
  10. skjnldsv commented on Nov 8, 2017

    @skjnldsv
    Member

    Can confirm on stable12!
    And actually, I can confirm on master13 too! ⚠️
    This only applies to the checkboxes in the popover.

    https://github.com/nextcloud/server/blob/e9ab452ff37f8ea606934bcc3323047d75d20e4a/core/js/sharedialogshareelistview.js

    @blizzz @schiessle ?

  11. assigned and unassigned on Nov 17, 2017
  12. blizzz commented on Nov 17, 2017

    @blizzz
    Member

    Fix in #7211

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

1. to developAccepted and waiting to be taken care ofbugdesignDesign, UI, UX, etc.feature: sharinggood first issueSmall tasks with clear documentation about how and in which place you need to fix things in.highregression

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions