Skip to content

Form Smart Paste: Implement Demos#30986

Merged
juleari merged 31 commits intoDevExpress:25_2from
juleari:25_2_smartpaste_demos
Sep 10, 2025
Merged

Form Smart Paste: Implement Demos#30986
juleari merged 31 commits intoDevExpress:25_2from
juleari:25_2_smartpaste_demos

Conversation

@juleari
Copy link
Contributor

@juleari juleari commented Sep 3, 2025

No description provided.

@juleari juleari self-assigned this Sep 3, 2025
@juleari juleari added the 25_2 label Sep 3, 2025
@juleari juleari requested a review from a team as a code owner September 3, 2025 12:43
type: 'normal'
};

smartPasteButtonOptions = {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
smartPasteButtonOptions = {
smartPasteButtonOptions: DxButtonTypes.Properties = {
stylingMode: 'contained',
type: 'default',
}

same for other frameworks

Copy link
Contributor

Choose a reason for hiding this comment

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

Let's add test scenario and compare 2 screenshots: initial and with loader

Copy link
Contributor Author

Choose a reason for hiding this comment

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

discussed, that it would be unwise to do so on demos, since this way we will spend tokens every time this tests will run, so we will need to mock AI request

created a card for this test

}

::ng-deep .buttons-group .dx-item {
flex: unset!important;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
flex: unset!important;
flex: unset !important;

@@ -0,0 +1,130 @@
<div id="form-container">
<div class="instruction" id="textarea-label"
>Copy text from the editor below to the clipboard. Edit the text to see how
Copy link
Contributor

Choose a reason for hiding this comment

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

Is all okay with formatting here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it was formatted by prettifier
pnpm run prettier --write Demos/Form/SmartPaste/Angular/app/app.component.html

}, {
itemType: 'group',
cssClass: 'buttons-group',
colCountByScreen,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
colCountByScreen,
colCount: 2,

Copy link
Contributor Author

Choose a reason for hiding this comment

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

with just colCount it will be one column on small screens, but by design we should have two columns everywhere

@juleari juleari force-pushed the 25_2_smartpaste_demos branch from 20096eb to 819be40 Compare September 9, 2025 11:16
@juleari juleari merged commit 81ff037 into DevExpress:25_2 Sep 10, 2025
357 of 358 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants