Skip to content

923649: For Creating a new add form depending on the NewRowPosition in MAC device, key code is changed #6371

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

Merged
merged 2 commits into from
Aug 13, 2025

Conversation

NandhiniLeninmoorthy
Copy link
Contributor

Bug Description

The key combination for creating new row in the MAC OS need to change.

Root Cause

Previous key combination not works and it perform another action, so need to change the key code for insert new row action for MAC.

Reason for not identifying earlier

Find how it was missed in our earlier testing and development by analysing the below checklist. This will help prevent similar mistakes in the future.

Guidelines/documents are not followed

Common guidelines / Core team guideline
Specification document
Requirement document

Guidelines/documents are not given

Common guidelines / Core team guideline
Specification document
Requirement document

Reason:
Mention any one or more reasons from the above points.

Action taken:
What action did you take to avoid this in future?

Related areas:

  1. Keyboard interaction.

Is Breaking issue?

No

Solution description

Changed the key code as ⌥ + Enter from ⌘ + Enter.

Output screenshots

Post the output screenshots if an UI is affected or added due to this bug.

Before changes:
image

After changes:
image

Areas affected and ensured

NA

Does it have any known issues?

If this fix contains any known issues, provide the proper details about the issues.

Does it have memory leak?

Ensure the fix contains memory leak or not (if applicable).

MR CheckList

  • Does it follow the design guidelines? It is mandatory that, we should not move the changes without reading this.
  • Did UI automation passed without errors? If it is customer issue, make sure it is included in the IR automation.
  • Properly working in Xamarin.Forms previewer.
  • Ensured in iOS, Android, UWP and macOS (if supported).
  • Have you ensured the changes in Android API 19 and iOS 9?
  • Did you record this case in the unit test or UI test?

@NandhiniLeninmoorthy NandhiniLeninmoorthy self-assigned this Aug 4, 2025
@NandhiniLeninmoorthy NandhiniLeninmoorthy added bug Something isn't working documentation Improvements or additions to documentation labels Aug 4, 2025
@SyncfusionBuild
Copy link
Contributor

Build Status: INPROGRESS 🔃
Build Location: 923649-InsertMac

@SyncfusionBuild
Copy link
Contributor

CI Status: FAILURE ❌
Source Compilation: SUCCESS ✅
Failure Reason: CI Failure with violation
Build Location: 923649-InsertMac
Technical Error(s): 0
Spelling Error(s): 0
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
Front matter Error(s): 1
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0
MD Tag Validation Error(s): 0

@SyncfusionBuild
Copy link
Contributor

Build Status: INQUEUE 🕒
Build Location: 923649-InsertMac

@SyncfusionBuild
Copy link
Contributor

Build Status: INPROGRESS 🔃
Build Location: 923649-InsertMac

@SyncfusionBuild
Copy link
Contributor

CI Status: SUCCESS ✅
Source Compilation: SUCCESS ✅
Build Location: 923649-InsertMac
Technical Error(s): 0
Spelling Error(s): 0
Image Alt Text Error(s): 0
Image Size Error(s): 0
Image Name Error(s): 0
Front matter Error(s): 0
File Path Error(s): 0
CodeSnippet File Path Missing Error(s): 0
Toc Anchor Tags Validation Error(s): 0
MD Tag Validation Error(s): 0

@NandhiniLeninmoorthy NandhiniLeninmoorthy changed the title 923649: For Creating a new add form depending on the NewRowPosition for MAC key code is changed 923649: For Creating a new add form depending on the NewRowPosition in MAC device, key code is changed Aug 5, 2025
Copy link
Collaborator

@dhivyasyncfusion dhivyasyncfusion left a comment

Choose a reason for hiding this comment

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

Hi @NandhiniLeninmoorthy , changes are fine

@dhivyasyncfusion dhivyasyncfusion merged commit 5cf8c1f into development Aug 13, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants