MaterialDesignInXAML / MaterialDesignInXAML/MaterialDesignInXamlToolkit

MaterialDesignDataGridTextColumnPopupEditingStyle: Mouse and focus issues

Open
#493 3 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

bug datagrid up-for-grabs
Dominant language
C#
Stars
16.3k
Forks
3.5k
Avg merge
1d 22h
Merged PRs (30d)
8

Description

When you double click on a cell which has `EditingElementStyle="{StaticResource MaterialDesignDataGridTextColumnPopupEditingStyle}"` defined, the popup appears but there are a few issues:

1) You can't move the cursor using the mouse and can't select text with the mouse (although right clicking brings up the context menu correctly).
2) If the cell editing popup is open and you change focus to another app, the popup remains on top of the other application.
3) (Minor/opinion) The cursor's position is at the beginning of the TextBox when the popup editor opens (I think this would typically be at the end?)

To reproduce:
- Open the WPF demo
- Selecting the "Grids" page using the menu
- Double click on a cell in the _Name_ or _Description_ columns to start editing
- Try using the left mouse button to move the cursor or select text inside the popup
- Switch to another application while the cell editor is open

Here's a screenshot of the popup over another window.
![popup_example](https://cloud.githubusercontent.com/assets/6824706/18284485/bc14b9d0-742f-11e6-8c24-ee7fdd55cbb8.png)

Contributor guide

Open the contributing guide

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.

Research direction

Start with the WPF demo's Grids page and the MaterialDesignDataGridTextColumnPopupEditingStyle used by the Name and Description columns. Reproduce the popup behavior with a double-click, then trace the popup editor's mouse selection, application-focus, and initial cursor-position handling. Done means mouse editing works, the popup does not stay above another application, and the cursor behavior is addressed.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
desktop
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.