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

Sheet: moving columns messes up sort/filter display #911

Open
jgeekmz opened this issue Aug 6, 2022 · 3 comments
Open

Sheet: moving columns messes up sort/filter display #911

jgeekmz opened this issue Aug 6, 2022 · 3 comments
Assignees
Labels
defect Something isn't working

Comments

@jgeekmz
Copy link

jgeekmz commented Aug 6, 2022

Describe the bug

Sheet with several columns with sort and filter options. When you add movable option for rows and columns. On one column you do not have sort and filter. So when switch the place on this one with other column with filter and sort the header is moved, but the sort and filter options stayed on the same place as before. I guess this behaviour is bug.

Stack Overflow: https://stackoverflow.com/questions/73248207/primefaces-sheet-component-movable-columns

Reproducer

<pe:sheet id="sheet" widgetVar="sheetWidget" value="#{vertragspreiseForm.lazyModel.getPreiseList()}" var="row"
        rowKey="#{row.pflegeartId}"
        height="400"
        resizableRows="true"
        resizableCols="true"
        fixedCols="2"
        showColumnHeaders="true"
        movableCols="true" movableRows="true">

Expected behavior

When you move a column i expect to move with sort and filter optons as well.

PrimeFaces Extensions version

7.0.3

JSF implementation

No response

JSF version

2.0

Browser(s)

Chrome

@jgeekmz jgeekmz added the defect Something isn't working label Aug 6, 2022
@melloware melloware changed the title Sheet Sheet: moving columns messes up sort/filter display Aug 6, 2022
@melloware
Copy link
Member

melloware commented Aug 6, 2022

Updated with Stack overflow info. My guess is these features were just never tested together as movable columns was added way after sorting/filtering.

@VsevolodGolovanov
Copy link

In a cellSelect listener sheet.getSelectedColumn() also reports wrong column index.

pe:sheet widget should do hot.toPhysicalColumn(col), because moving columns changes their visual indexes (which is what most of HOT's API uses).

@melloware
Copy link
Member

PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants