Skip to content

Add PrintDialogDesigner.cs#9579

Closed
LeafShi1 wants to merge 16 commits intodotnet:feature/9.0from
LeafShi1:Issue_1908_Port_PrintDialogDesigner
Closed

Add PrintDialogDesigner.cs#9579
LeafShi1 wants to merge 16 commits intodotnet:feature/9.0from
LeafShi1:Issue_1908_Port_PrintDialogDesigner

Conversation

@LeafShi1
Copy link
Copy Markdown
Member

@LeafShi1 LeafShi1 commented Jul 27, 2023

Related to #4908

Proposed changes

  • Port MonthCalendarDesigner to runtime

Customer Impact

  • PrintDialogDesigner can be designed in runtime

Regression?

  • No

Risk

  • Minimal

Screenshots

Before

The default value of the property "UseEXDialog" is false
image

After

The default value of the property "UseEXDialog" is true
image

Test methodology

  • Manually (added PrintDialogDesigner to DemoConsole test app)

Test environment(s)

  • .Net 8.0.100-rc.1.23375.11
  • Windows 10 Enterprise 22H2
Microsoft Reviewers: Open in CodeFlow

dreddy-work and others added 8 commits July 25, 2023 16:29
* Move BindToObject to a separate file

* Enable nullability in BindToObject

* Use pattern matching

* Make lManager nullable

* Make value nullable

* Use pattern matching and remove text variable
[main] Update dependencies from dotnet/runtime
* Update language format from ll_CC to LCID

Related to #9556

* Update xlfs.

* Update back to ll-CC format with right syntax suggested by loc.

* update xlf
* Rework scrolling in PrintPreviewControl:

- replace native scrollbars with HScrollBar and VScrollBar controls;
- draw focus rectangle
- modify UIA tree to include new scrollbars.

* Reverted hiding PrintPreviewControl.TabStop property from Designer. Default value is left 'false'.

* Code cleanup:

- renamed fields to start with underscore;
- reordered class members to improve readability;
- refactored size calculation methods to avoid unnecessary casts.

* PR feedback:

- moved PrintPreviewControlAccessibleObject file to Printing folder;
- reverted adding SR resource strings;
- added FragmentNavigate method for PrintPreviewControlAccessibleObject;
- added custom AccessibleObject classes for scrollbars, to maintain UIA tree integrity;
- added unit tests.

* PR feedback:�- set scrollbars' TabStop to false

* Fixed: focus rectangle not visible when no document is previewed.

* Fixed PublicAPI.Shipped.txt
@LeafShi1 LeafShi1 requested a review from a team as a code owner July 27, 2023 07:57
@ghost ghost assigned LeafShi1 Jul 27, 2023
@LeafShi1 LeafShi1 changed the base branch from main to feature/9.0 July 27, 2023 07:58
SimonZhao888
SimonZhao888 previously approved these changes Jul 27, 2023
@elachlan
Copy link
Copy Markdown
Contributor

@LeafShi1 this PR contains commits which aren't related to the issue. When you rebase, you might need to "Skip" some of the commits from the previous "base". I ran into a similar issue.

Copy link
Copy Markdown
Member

@dreddy-work dreddy-work left a comment

Choose a reason for hiding this comment

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

Please rebase.

@ghost ghost added the waiting-author-feedback The team requires more information from the author label Jul 28, 2023
@LeafShi1 LeafShi1 changed the base branch from feature/9.0 to main July 28, 2023 06:55
@LeafShi1 LeafShi1 dismissed SimonZhao888’s stale review July 28, 2023 06:55

The base branch was changed.

@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Jul 28, 2023
LeafShi1 and others added 7 commits July 27, 2023 23:58
* Port MonthCalendarDesigner to runtime

* Delete unnecessary namespace

* Delete redundant assertions and redundant judgment conditions

* Update MonthCalendarDesigner.cs to use file-scoped namespaces
Fixes #9464
* Fix Cursor.ToString - do not look up user-provided cursors in the well known list.
* Add test 
* Fix WinformsControlsTest not showing System.Drawing.Design editors
…ayed in properties window #5337 (#9570)

Fix items property for multiple checked list boxes should not be displayed in properties window #5337
* Implement ImageListDesigner

* Refactoring ImageListActionList

* Move AssemblyRef to common

* Changes from review

* Few missed changes

* Revert "Move AssemblyRef to common" and add required changes

---------

Co-authored-by: Devendar Reddy Adulla <dreddy@microsoft.com>
@LeafShi1 LeafShi1 changed the base branch from main to feature/9.0 July 28, 2023 07:07
@LeafShi1
Copy link
Copy Markdown
Member Author

  • .Net 8.0.100-rc.1.23375.11
  • Windows 10 Enterprise 22H2

@LeafShi1 this PR contains commits which aren't related to the issue. When you rebase, you might need to "Skip" some of the commits from the previous "base". I ran into a similar issue.

Thanks for your reminder! After I rebase, it still included the extra commit, I submitted a new PR and deleted this one.

@LeafShi1 LeafShi1 closed this Jul 28, 2023
@LeafShi1 LeafShi1 deleted the Issue_1908_Port_PrintDialogDesigner branch July 28, 2023 08:48
@ghost ghost locked as resolved and limited conversation to collaborators Aug 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants