Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request focuses on improving UI/UX consistency and usability in the AXOpen.Data.Blazor data exchange components. The changes primarily involve button style standardization, UI reorganization for better clarity, and refactoring for improved maintainability.
Changes:
- Standardized button styles across data exchange views by replacing inconsistent CSS classes (e.g.,
btn-outline-red→btn-outline-primaryorbtn-outline-danger) and updating icon styling - Enhanced UI structure with semantic data attributes for better testability and clearer organization of filtering/operations sections
- Improved user experience by displaying sort member names directly in buttons, adding localized text for sorting direction, and repositioning filter controls
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
src/styling/src/wwwroot/css/momentum.css |
Added new utility classes (size-7, size-8, m-0!) to support UI updates |
src/data/src/AXOpen.Data.Blazor/Distributed/DistributedDataView.razor |
Updated button styles, added data attributes, restructured layout with flex properties, changed count display markup |
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/Query/Symbol/QuerySymbolConfigurationView.razor |
Replaced dropdown component with native select element for operation selection, added AccessStatusClass parameter support |
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/Query/Symbol/DynamicInput.razor |
Added AccessStatusClass parameter and propagated it to input components for consistent styling |
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/Query/DataExchangeQuery.razor.cs |
Enhanced query history loading logic with improved null/empty checks and fallback behavior |
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/Query/DataExchangeQuery.razor |
Reorganized filter UI, swapped button order, updated styles to btn-outline-primary, changed button labels |
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/DataExchangeView.razor.cs |
Renamed method from setSortAscendingAsync to ToggleSortSettings for clarity |
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/DataExchangeView.razor |
Standardized button styles, updated labels ("Create" → "Create new"), changed icon types and sizes, improved sort display with member name |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/Query/DataExchangeQuery.razor
Outdated
Show resolved
Hide resolved
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/DataExchangeView.razor
Outdated
Show resolved
Hide resolved
src/data/src/AXOpen.Data.Blazor/Distributed/DistributedDataView.razor
Outdated
Show resolved
Hide resolved
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/DataExchangeView.razor
Outdated
Show resolved
Hide resolved
src/data/src/AXOpen.Data.Blazor/AxoDataExchange/Query/Symbol/DynamicInput.razor
Outdated
Show resolved
Hide resolved
…angeQuery.razor Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ub.com/Inxton/AXOpen into 969-bug-distributeddata---globalfilter
src/data/src/AXOpen.Data.Blazor/Distributed/DistributedDataView.razor
Dismissed
Show dismissed
Hide dismissed
src/data/src/AXOpen.Data.Blazor/Distributed/DistributedDataView.razor
Dismissed
Show dismissed
Hide dismissed
LukasKytka
approved these changes
Feb 24, 2026
Brano5
approved these changes
Feb 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request focuses on improving the UI/UX consistency and clarity in the
AXOpen.Data.Blazordata exchange components. The main changes involve updating button styles and labels for better user experience, reorganizing UI elements for clarity, and making minor code refactors for maintainability.UI/UX Improvements:
btn-outline-redtobtn-outline-primaryorbtn-outline-danger) and aligning icon usage for action buttons. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]UI Structure and Filtering Enhancements:
data-dataexchange-operations,data-filtering-operations) to key UI sections for better structure and potential testability. [1] [2]DataExchangeQuery.razorby repositioning the filter input, adjusting gap and margin for compactness, and swapping the order of "Clean" and "All" buttons for logical flow.Sorting and Filtering Logic:
setSortAscendingAsyncmethod with the more descriptiveToggleSortSettings, and updated the sorting toggle button to display localized "Ascending"/"Descending" text instead of just an icon. [1] [2]Code Cleanliness:
Button/Icon Consistency:
These changes collectively enhance the usability, maintainability, and visual consistency of the data exchange UI components.