Skip to content

Add VS4Mac/macOS pivot to MAUI-Blazor tutorial#27678

Merged
guardrex merged 3 commits into
mainfrom
guardrex/blazor-hybrid-maui-tutorial-mac
Nov 19, 2022
Merged

Add VS4Mac/macOS pivot to MAUI-Blazor tutorial#27678
guardrex merged 3 commits into
mainfrom
guardrex/blazor-hybrid-maui-tutorial-mac

Conversation

@guardrex
Copy link
Copy Markdown
Collaborator

@guardrex guardrex commented Nov 18, 2022

Addresses #26364

Internal Review Topic

  • This is in pretty much 🦖 Rex Pea-brain Dump™ 🦖 state due to the speed that I cranked it out this morning 🏃‍♂️. It's a bit rough, but I think it's ok to take feedback on it in this state given that I'm about out of 🕦 this week. I'll try to address feedback today (Friday) if it arrives. I'll review/edit/polish on Monday morning either way.

  • Check the pivot selector prompt text ("Choose a Visual Studio operating system", aspnetcore/zone-pivot-groups.yml) to see what you want that to say.

  • I placed an IMPORTANT note about the IDE install order and what to do if it's accidentally performed in reverse. I did it in reverse by mistake 🙈 and hit the MIA SDK error, so I'd like to head off doc repo issues by leaving this guidance in here somewhere. It could go at the end with a cross-link from the top or in the MAUI tutorial overview topic with a cross-link from the top ... or where I have it now on the PR.

  • I had a warning here about the Java SDK, but it didn't affect this scenario. I haven't researched it, so idk what's what with it.

    warning
  • WRT the WinOS pivot: I sent the remarks about the WinOS problems in the offline chat. I think that the Dock > Fill problem was present back in the 6.0 release, too. The component reference problem goes away after the app is built the first time.

@guardrex guardrex mentioned this pull request Nov 18, 2022
8 tasks
@guardrex guardrex self-assigned this Nov 18, 2022
Comment thread aspnetcore/blazor/hybrid/tutorials/maui.md
@danroth27
Copy link
Copy Markdown
Member

Check the pivot selector prompt text ("Choose a Visual Studio operating system", aspnetcore/zone-pivot-groups.yml) to see what you want that to say.

The current text seems fine.

I placed an IMPORTANT note about the IDE install order and what to do if it's accidentally performed in reverse. I did it in reverse by mistake 🙈 and hit the MIA SDK error, so I'd like to head off doc repo issues by leaving this guidance in here somewhere. It could go at the end with a cross-link from the top or in the MAUI tutorial overview topic with a cross-link from the top ... or where I have it now on the PR.

The note and its location look reasonable to me.

I had a warning here about the Java SDK, but it didn't affect this scenario. I haven't researched it, so idk what's what with it.

Sounds like something we should follow up on, but it doesn't block this doc. Can you create a VS feedback issue for this and share the link with us?

WRT the WinOS pivot: I sent the remarks about the WinOS problems in the offline chat. I think that the Dock > Fill problem was present back in the 6.0 release, too. The component reference problem goes away after the app is built the first time.

These are problems with the Windows Forms experience, right? I'll follow up with you on these separately.

@guardrex
Copy link
Copy Markdown
Collaborator Author

These are problems with the Windows Forms experience, right? I'll follow up with you on these separately.

Yes, the screenshots for those two are in that email that you responded to yesterday. The subject line is "Almost there on Blazor docs 7.0 coverage". It's the first message of that chain.

Can you create a VS feedback issue for this and share the link with us?

Yes. Will do. 👍

I'll need to get back to you tho in a few hours on getting back on the MacBook.

I'll get back to you in a few hours on that, too.

@guardrex
Copy link
Copy Markdown
Collaborator Author

guardrex commented Nov 18, 2022

@danroth27 ...

Should I go ahead and publish this now?

WRT the two reports that I need to generate for the WinForms-Blazor app (Dock > Fill and Counter component squiggle), I'll do that first thing Saturday morning, as I'm about to go OOF again in a few minutes. I'll send those two links via email tomorrow morning.

I'm about to place the link to the Java SDK Dev Community issue into the email chain. Done! 👍

Copy link
Copy Markdown
Member

@danroth27 danroth27 left a comment

Choose a reason for hiding this comment

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

LGTM

@guardrex guardrex merged commit 37db016 into main Nov 19, 2022
@guardrex guardrex deleted the guardrex/blazor-hybrid-maui-tutorial-mac branch November 19, 2022 11:18
Donciavas pushed a commit to Donciavas/AspNetCore.Docs that referenced this pull request Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants