magento / magento/community-features

Allow to set negative number for attribute position in layered navigation

Open
#269 3 comments 0 reactions 1 assignee View on GitHub

@ravi-chandra3197 is already working on this.

Since Dec 23, 2019.

Priority: P4 Progress: ready for dev Severity: S4
Dominant language
No language data
Stars
46
Forks
14
PR merge metrics
No merged PRs in 30d

Description

### Description (*)

As _catalog_eav_attribute_ table position is an integer, it is legal and seems completely safe to set a negative position for any attribute in layered navigation. I would like to replace attribute position validation from _validate-digits_ to _integer_.

### Expected behavior (*)

Admin attribute edit should allow to set any positive or negative non-decimal number for position in layered navigation.

### Benefits

That would allow to prioritize chosen attributes more easily.
### Additional information

![negative-position-of-attribute](https://user-images.githubusercontent.com/5328659/69839818-70ad0d00-1259-11ea-9589-ffc266d872a2.png)

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.