Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-25229

Disable the move button for Location which depth is 1

    Details

      Description

      Actually it seems to be possible to move root location to another one. But if you do that, you get a Failed to load root locations and you cannot browse content tree anymore.

        Issue Links

          Activity

          Hide
          Damien Pobel (Inactive) added a comment -

          Indeed that's a bit stupid I guess the same is valid for the "Media Library" and the "Users" Locations. And somehow, we most likely have the same kind of issue with "Trash" as well.

          [~jince.kuruvilla@ez.no] Roland Benedetti Inaki Juaniz-Velilla And then the same question as in EZP-25209 arises, do we completely hide the button or do we apply a "disabled" style to the "Move" buttons ?

          Show
          Damien Pobel (Inactive) added a comment - Indeed that's a bit stupid I guess the same is valid for the "Media Library" and the "Users" Locations. And somehow, we most likely have the same kind of issue with "Trash" as well. [~jince.kuruvilla@ez.no] Roland Benedetti Inaki Juaniz-Velilla And then the same question as in EZP-25209 arises, do we completely hide the button or do we apply a "disabled" style to the "Move" buttons ?
          Hide
          Carlos Revillo added a comment -

          i'd vote por hide. but not only hide by css but remove it completely from the dom...

          Show
          Carlos Revillo added a comment - i'd vote por hide. but not only hide by css but remove it completely from the dom...
          Hide
          Jince Kuruvilla (Inactive) added a comment -

          I would disable the button here as well.

          We currently have the luxury of having lots of breathing room in the action drawer right now, but our decision on this matter might change once space becomes an issue (and we might just hide the button altogether as Carlos suggested) but for now, let's just disable.

          Show
          Jince Kuruvilla (Inactive) added a comment - I would disable the button here as well. We currently have the luxury of having lots of breathing room in the action drawer right now, but our decision on this matter might change once space becomes an issue (and we might just hide the button altogether as Carlos suggested) but for now, let's just disable.
          Hide
          Damien Pobel (Inactive) added a comment -

          Renamed the issue to "Disable the move button for Location which depth is 1" so that we disable the button the "Content Structure", "Media Libary" and "Users" locations.

          Show
          Damien Pobel (Inactive) added a comment - Renamed the issue to "Disable the move button for Location which depth is 1" so that we disable the button the "Content Structure", "Media Libary" and "Users" locations.
          Show
          Mateusz Hyndle (Inactive) added a comment - - edited PR: https://github.com/ezsystems/PlatformUIBundle/pull/475 merged in master: https://github.com/ezsystems/PlatformUIBundle/commit/e0283c6316c3a2140bffcf68f2487d85d3ee0eb9
          Hide
          Miguel das Neves Jacinto (Inactive) added a comment -

          QA Approved

          Show
          Miguel das Neves Jacinto (Inactive) added a comment - QA Approved
          Hide
          Bertrand Dunogier added a comment -

          Cherry-picked to 1.0 in https://github.com/ezsystems/PlatformUIBundle/commit/394e4f8 (ezsystems/platform-ui-bundle:v1.0.1).

          Show
          Bertrand Dunogier added a comment - Cherry-picked to 1.0 in https://github.com/ezsystems/PlatformUIBundle/commit/394e4f8 (ezsystems/platform-ui-bundle:v1.0.1).

            People

            • Assignee:
              Unassigned
              Reporter:
              Carlos Revillo
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 7 hours, 15 minutes
                7h 15m