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

Admin Category Management UI issues #664

Closed
tzyganu opened this issue Sep 12, 2014 · 4 comments
Closed

Admin Category Management UI issues #664

tzyganu opened this issue Sep 12, 2014 · 4 comments
Assignees

Comments

@tzyganu
Copy link
Contributor

tzyganu commented Sep 12, 2014

I found some issues related to the admin UI for managing catalog categories.
I added all of them in a single issue. (I already "spammed" enough this repo).

Issue 1
The inactive categories are not grey-ed out in the tree. So there is no difference between the active categories and inactive ones.
disabled

Issue 2
Deleting a category right after creating it actually deletes the parent category. Here is how you can reproduce it.

  1. Add a child category to one of the existing categories (will call it "parent")

  2. Immediately after creating the category, you realize you created it wrongly and delete it.

  3. Click on Delete category.

  4. The parent category is deleted (along with the one I actually wanted to delete).

    It even gets worse. Let's say I have one category. Will call it again "parent". and I want to create the following tree structure all in one process. I mean as soon as I add one child category I add a child to that child.


      parent - (already existing)  
         |-- child level 1 - I'm creating this one  
                    |-- child level 2 - Created imediately after child level 1  
                                 |-- child level 3 - Created imediately after child level 2  

And now I want to delete "child level 3". What actually happens, is "parent" gets deleted with all the child categories.
The issue does not reproduce if I reload the page after each "add" action.

Issue 3
After adding a child category the title section of the page still has the parent category value.
Before I took the screenshot below I did this:
1. Clicked on "Household Items" category
2. Clicked "Add subcategory"
3. Created the "Child Category" category.
4. Save.
But if I scroll down on the page until the floating header appears, in the floating header I see the corect category title.
difference

Issue 4
When moving a category in the tree, there is no loader overlay displayed. I think this can lead to problems if the user does any actions until the move process is finished

Issue 5
I created a bunch of categories as children of the root catalog category. then added one as a child for an other until I got to have the biggest level as 10.
When clicking on the menu Products->Categories I don't have the option to expand categories with the level 6 or higher. I only see the categories with level higher than 6 if I select for edit one of the level 6 categories.

This is what it appears after I click Products->Categories and Expand all or after manually expanding all I can expand
manual

This is my full category tree. I get it only after I click in the tree above on the categories with big level number: (In the case above Hoodies then 'Child Category' then 'Downloadable')
full

@verklov
Copy link
Contributor

verklov commented Sep 23, 2014

@tzyganu, thanks for a new bunch of defects! We created tickets in internal bug tracker for each of the points. However, please report each of them in an individual issue. This will make tracking issues and processing them easier.

@tzyganu
Copy link
Contributor Author

tzyganu commented Sep 24, 2014

@verklov OK I got it. I thought that since you have an internal bug tracker it would not make any difference if I report related issues all in the same place. You want me to split this into 5 different issues now, or only from now on?

@verklov
Copy link
Contributor

verklov commented Sep 29, 2014

@tzyganu, to track each issue properly I as a PO enter each individual ticket reported on the GitHub to the internal bug tracker. This also helps me in reporting. You do not need to split the already reported tickets. Please do this from now on. Thanks!

@maksek maksek added PROD and removed major-check labels Jan 8, 2015
@Tiurnikova
Copy link

Hi @tzyganu
We a re planning to update the Category tree. There is a ticket in our internal backlog now to test and fix all the reported issues. Thank you so much!

paliarush pushed a commit that referenced this issue Oct 2, 2015
[API] Sprint 58 Bug Fixes and Test Coverage
okorshenko pushed a commit that referenced this issue Feb 7, 2017
magento-engcom-team added a commit that referenced this issue Mar 20, 2019
Accepted Public Pull Requests:
 - #21785: [Forwardport] [Checkout] Fix clearing admin quote address when removing all items (by @eduard13)
 - #21469: Update price-bundle.js so that correct tier price is calculated while displaying in bundle product (by @adarshkhatri)
 - #21751: fix #21750 remove translation of product attribute label (by @Karlasa)
 - #21774: MSI: Add deprecation message to CatalogInventory SPIs (by @lbajsarowicz)
 - #21575: Fix for issue #21510: Can't access backend indexers page after creating a custom index (by @ccasciotti)
 - #21288: #12396: Total Amount cart rule without tax (by @AleksLi)
 - magento-commerce/magento-functional-tests-migration#679: #417 Removed unnecessary default values in action groups (by @hws47a)
 - magento-commerce/magento-functional-tests-migration#678: Convert FlushStaticFilesCacheButtonVisibilityTest to MFTF (by @Leandry)
 - magento-commerce/magento-functional-tests-migration#676: Convert VerifyDisabledCustomerGroupFieldTest to MFTF #664 (by @Michalk39)


Fixed GitHub Issues:
 - #21467: Tier price of simple item not working in Bundle product (reported by @adarshkhatri) has been fixed in #21469 by @adarshkhatri in 2.3-develop branch
   Related commits:
     1. caabef2
     2. 4c1c653
     3. d4dd2e6
     4. e7589ad
     5. 6f49b07
     6. 0332391

 - #21750: Product attribute labels are translated (reported by @Karlasa) has been fixed in #21751 by @Karlasa in 2.3-develop branch
   Related commits:
     1. a07eda3

 - #21510: Can't access backend indexers page after creating a custom index (reported by @ccasciotti) has been fixed in #21575 by @ccasciotti in 2.3-develop branch
   Related commits:
     1. 318425f
     2. 486e21c
     3. 409e00e
     4. 0acd6b6

 - #12396: "Total Amount" cart rule without tax (reported by @Chei20) has been fixed in #21288 by @AleksLi in 2.3-develop branch
   Related commits:
     1. 6ae859a
     2. c25656b
     3. 23ed015
     4. e9389ee
     5. 571f055
magento-engcom-team added a commit that referenced this issue Aug 13, 2019
 - Merge Pull Request magento/graphql-ce#664 from pmclain/graphql-ce:issue/633
 - Merged commits:
   1. c8e30b8
   2. d26dbbc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants