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

Create button seems available even if it's not

    Details

      Description

      The button for creating new content is displayed even if the content type is set not to be a container. The pop-out displaying content types, however, appears to never load.

        Activity

        Hide
        Damien Pobel (Inactive) added a comment -

        Well, actually the is container flag is taken into account and that's why nothing happens when you click on it when you are displaying a non container content in PlatformUI. That's said, I agree with you it's not super clear for the end user as there's no visual difference on the button in that case.

        We can either completely hide the button or add a style so that the user sees it's not available in the current context. In such case, I would choose the later strategy (and for that we just have to add a few lines of CSS actually).

        [~jince.kuruvilla@ez.no] Roland Benedetti Inaki Juaniz-Velilla I remember we had the same discussion after a demo some times ago, any opinion on that ?

        Show
        Damien Pobel (Inactive) added a comment - Well, actually the is container flag is taken into account and that's why nothing happens when you click on it when you are displaying a non container content in PlatformUI. That's said, I agree with you it's not super clear for the end user as there's no visual difference on the button in that case. We can either completely hide the button or add a style so that the user sees it's not available in the current context. In such case, I would choose the later strategy (and for that we just have to add a few lines of CSS actually). [~jince.kuruvilla@ez.no] Roland Benedetti Inaki Juaniz-Velilla I remember we had the same discussion after a demo some times ago, any opinion on that ?
        Hide
        Jince Kuruvilla (Inactive) added a comment -

        [~damien.pobel@ez.no] My vote is for a grayed out button showing that it exists, but is unavailable in the current context.

        Show
        Jince Kuruvilla (Inactive) added a comment - [~damien.pobel@ez.no] My vote is for a grayed out button showing that it exists, but is unavailable in the current context.
        Show
        Damien Pobel (Inactive) added a comment - PR: https://github.com/ezsystems/PlatformUIBundle/pull/469
        Show
        Damien Pobel (Inactive) added a comment - Merged in master https://github.com/ezsystems/PlatformUIBundle/commit/deb02a977b8d69bf08adb410f2ca06fe8da0e78e
        Hide
        Rui Silva (Inactive) added a comment -

        Tested and approved by QA for master.

        Show
        Rui Silva (Inactive) added a comment - Tested and approved by QA for master.

          People

          • Assignee:
            Unassigned
            Reporter:
            Eirik Alfstad Johansen
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: