Skip to content

Conversation

@JuliRossi
Copy link
Collaborator

@JuliRossi JuliRossi commented Aug 29, 2025

Purpose

We want to be able to filter columns for bulk edit.

Approach

In this PR:

  • Add filter capability
  • Improve state management for content Type
  • Fix box warning from sticking properties (because it was affecting the filter columns )

Testing steps

Grabacion.de.pantalla.2025-08-29.a.la.s.10.34.12.a.m.mov

Deployment

We want to merge this after V1.1 is merge to master

@JuliRossi JuliRossi added the do not merge Don't merge this PR until this label is removed label Aug 29, 2025
@JuliRossi JuliRossi changed the base branch from bulk-edit-v1.2 to bulk-edit-v1.1 August 29, 2025 13:10
@JuliRossi JuliRossi changed the title Bulk edit integ 3089 Bulk edit: Filter columns [INTEG-3089] Aug 29, 2025
@JuliRossi JuliRossi marked this pull request as ready for review August 29, 2025 13:32
@JuliRossi JuliRossi requested a review from a team as a code owner August 29, 2025 13:33
Base automatically changed from bulk-edit-v1.1 to master September 2, 2025 11:32
@JuliRossi JuliRossi force-pushed the bulk-edit-integ-3089 branch from ffaedef to 18b14fa Compare September 2, 2025 12:59
@netlify
Copy link

netlify bot commented Sep 2, 2025

Deploy Preview for ecommerce-app-base-components canceled.

Name Link
🔨 Latest commit 18b14fa
🔍 Latest deploy log https://app.netlify.com/projects/ecommerce-app-base-components/deploys/68b6ea1d32fd71000840b19b

@JuliRossi JuliRossi removed the do not merge Don't merge this PR until this label is removed label Sep 2, 2025
@JuliRossi JuliRossi changed the base branch from master to bulk-edit-v1.2 September 2, 2025 12:59
Copy link
Collaborator

@FBanfi FBanfi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great update!! Left some minor comments! 🚀

@FBanfi
Copy link
Collaborator

FBanfi commented Sep 2, 2025

Great update!! Left some minor comments! 🚀

Did the corrections since Juli was working in the AI guidelines doc

@FBanfi FBanfi force-pushed the bulk-edit-integ-3089 branch from 83d9edb to 86cbeff Compare September 3, 2025 13:30
@JuliRossi JuliRossi merged commit b4b0af4 into bulk-edit-v1.2 Sep 3, 2025
11 checks passed
@JuliRossi JuliRossi deleted the bulk-edit-integ-3089 branch September 3, 2025 13:48
@JuliRossi JuliRossi mentioned this pull request Sep 3, 2025
JuliRossi added a commit that referenced this pull request Sep 4, 2025
* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* fixing view numbers (#10092)

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
JuliRossi added a commit that referenced this pull request Sep 11, 2025
* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
JuliRossi added a commit that referenced this pull request Sep 17, 2025
* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
JuliRossi added a commit that referenced this pull request Sep 17, 2025
* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
JuliRossi added a commit that referenced this pull request Sep 25, 2025
* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
JuliRossi added a commit that referenced this pull request Oct 21, 2025
* Keyboard accesibility for bulk edit [MAPS-29] (#10110)

* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* Tooltip fix for bulk edit [MAPS-57] (#10148)

* fix

* wip

* Workaround to make both things work

* disabling filter when entries are loading (#10150)

* Bulk Edit: Improved performance [MAPS-72] (#10163)

* Scroll when navigating with keyboard

* Add virtual scrolling

* Adjust styles

---------

Co-authored-by: JuliRossi <juliana.rossi@external.contentful.com>
Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
FBanfi added a commit that referenced this pull request Oct 21, 2025
* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
FBanfi added a commit that referenced this pull request Oct 21, 2025
* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>
FBanfi added a commit that referenced this pull request Oct 21, 2025
)

* Keyboard accesibility for bulk edit [MAPS-29] (#10110)

* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* Tooltip fix for bulk edit [MAPS-57] (#10148)

* fix

* wip

* Workaround to make both things work

* adding more field editors

* adding test for entry utils content type mapper

* refactors + removing createLocales

* parsing the value for the different editors

* setting truncation to display fields

* adding more tests

* pr comments

* format changes between conflicts resolution

* changing utils file name

---------

Co-authored-by: JuliRossi <juliana.rossi@external.contentful.com>
FBanfi added a commit that referenced this pull request Oct 21, 2025
…10159)

* Keyboard accesibility for bulk edit [MAPS-29] (#10110)

* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* Tooltip fix for bulk edit [MAPS-57] (#10148)

* fix

* wip

* Workaround to make both things work

* changing some icons to match field editors versions + adding initial editors

* changing icon + modifying tests

* adding tests for FieldEditor component

* adding more field editors

* adding test for entry utils content type mapper

* renaming method

* removing unused variable

* moving methods to entry utils file

* removing any

* removing unnecessary logic for field editor setter

* changing disabled logic

* refactors + removing createLocales

* using field locale instead of default

* removing manual numeric validations

* removing mock from BulkEditModal.test.tsx

* fixing tests between conflicts

* Bulk-Edit: Parsing the value for the different editors [MAPS-53] (#10173)

* Keyboard accesibility for bulk edit [MAPS-29] (#10110)

* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* Tooltip fix for bulk edit [MAPS-57] (#10148)

* fix

* wip

* Workaround to make both things work

* adding more field editors

* adding test for entry utils content type mapper

* refactors + removing createLocales

* parsing the value for the different editors

* setting truncation to display fields

* adding more tests

* pr comments

* format changes between conflicts resolution

* changing utils file name

---------

Co-authored-by: JuliRossi <juliana.rossi@external.contentful.com>

---------

Co-authored-by: JuliRossi <juliana.rossi@external.contentful.com>
joaquincasal added a commit that referenced this pull request Nov 11, 2025
* Bulk-Edit: field editors for initial default appearances [MAPS-53] (#10159)

* Keyboard accesibility for bulk edit [MAPS-29] (#10110)

* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* Tooltip fix for bulk edit [MAPS-57] (#10148)

* fix

* wip

* Workaround to make both things work

* changing some icons to match field editors versions + adding initial editors

* changing icon + modifying tests

* adding tests for FieldEditor component

* adding more field editors

* adding test for entry utils content type mapper

* renaming method

* removing unused variable

* moving methods to entry utils file

* removing any

* removing unnecessary logic for field editor setter

* changing disabled logic

* refactors + removing createLocales

* using field locale instead of default

* removing manual numeric validations

* removing mock from BulkEditModal.test.tsx

* fixing tests between conflicts

* Bulk-Edit: Parsing the value for the different editors [MAPS-53] (#10173)

* Keyboard accesibility for bulk edit [MAPS-29] (#10110)

* Bulk-Edit-App: Fix sorting error and edit button with no padding [INTEG-3103] (#10090)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* not showing the edit button when loading entries

* fix error when changing sorting

* Bulk edit: Filter columns [INTEG-3089] (#10089)

* Bulk-Edit-App: Freeze top row with Field Names [INTEG-2953] (#10076)

* freeze top row with Field Names

* removing unused import

* making the status column sticky too (#10082)

* wip

* select all

* Refactor FilterColumns and SortMenu components for improved layout and functionality

* Fixing states and enhancing performance in the process by not calling the getContentType each time

* Fix box issue

* Renaming and fixing warnings

* sticky

* corrections PR comments

* Fixing rebase conflicts

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* wip

* Added new useKeyboardNavigation hook to encapsulate keyboard navigation logic.

* Simplifying a bit

* Refactor keyboard navigation logic in useKeyboardNavigation hook for improved readability and performance. Simplified moveFocus and extendFocusToEdge functions by removing unnecessary useCallback and enhancing selection handling.

* Refactor EntryTable and TableHeader components to improve keyboard navigation and selection handling. Updated focus logic to use HEADERS_ROW constant for better readability and maintainability. Enhanced checkbox toggle functionality for header and row selections.

* fixing issues

* changing styles for keyboard accessibility

* Refactors and tests

* Fixing focus on first cell and edge navigation

* Readding column selection

* Refactoring styles

* Refactor Table components to centralize cell focus and selection logic.

* Fixing checked disable checkboxes

* Refactor EntryTable, TableHeader, and TableRow components to unify cell focus and selection logic. Updated function signatures to use FocusPosition for better clarity and maintainability. Enhanced keyboard navigation handling in useKeyboardNavigation hook and corresponding tests.

* Simplifying a few things. Enter doesn't do that much anymore

* fix merge

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: francobanfi <franco.banfi@external.contentful.com>

* Tooltip fix for bulk edit [MAPS-57] (#10148)

* fix

* wip

* Workaround to make both things work

* adding more field editors

* adding test for entry utils content type mapper

* refactors + removing createLocales

* parsing the value for the different editors

* setting truncation to display fields

* adding more tests

* pr comments

* format changes between conflicts resolution

* changing utils file name

---------

Co-authored-by: JuliRossi <juliana.rossi@external.contentful.com>

---------

Co-authored-by: JuliRossi <juliana.rossi@external.contentful.com>

* Bulk edit: Refactor display names [MAPS-78] (#10205)

* refactor to reuse existing display value method

* refactor for getFieldDisplayValue method

* refactor in the getEntryFieldValue method

* removing test

* Bulk Edit: Customize the Boolean editor label with user settings [MAPS-86] (#10210)

* first version of the customizable boolean labels

* refactor to use useSdk hook

* adding tests and some refactors

* fixing pr comments

* removing unused imports

* Bulk Edit: Alternative appearances [MAPS-54] (#10219)

* first version of the customizable boolean labels

* refactor to use useSdk hook

* fixing pr comments

* removing unused imports

* adding alternative appearances for field editors

* Bulk edit: Add field validations [MAPS-52] (#10221)

* Add field validations

* Address PR comments

- Remove duplicated debounced value
- Updated the way error messages are displayed so they're consistent with errors from the field editors
- Removed redundant isValid property

* Bulk edit v2 fixes [MAPS-22] (#10229)

* Run validations for items

* Remove value from update if empty

---------

Co-authored-by: Franco Banfi <62450599+FBanfi@users.noreply.github.com>
Co-authored-by: JuliRossi <juliana.rossi@external.contentful.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants