Skip to content
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

Add platform to glossary #30731

Closed
wants to merge 1 commit into from
Closed

Conversation

jbogarthyde
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

"Platform" term not in glossary

Issue Number: #27096

What is the new behavior?

Adds "platform" term with ref to API doc and Browser Support page.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

API doc for PlatformRef/PlatformBrowser is not adequate, and there is no usage doc except a detached cookbook page, https://angular.io/guide/set-document-title

@jbogarthyde jbogarthyde requested a review from a team as a code owner May 29, 2019 16:40
@jbogarthyde jbogarthyde added comp: docs effort1: hours target: patch This PR is targeted for the next patch release type: bug/fix labels May 29, 2019
@ngbot ngbot bot modified the milestone: needsTriage May 29, 2019
@mary-poppins
Copy link

You can preview b936e4e at https://pr30731-b936e4e.ngbuilds.io/.

@mary-poppins
Copy link

You can preview b9be57d at https://pr30731-b9be57d.ngbuilds.io/.

aio/content/guide/glossary.md Outdated Show resolved Hide resolved
aio/content/guide/glossary.md Outdated Show resolved Hide resolved
aio/content/guide/glossary.md Show resolved Hide resolved
@jbogarthyde jbogarthyde requested a review from a team as a code owner May 29, 2019 22:29
@mary-poppins
Copy link

You can preview 242c7b2 at https://pr30731-242c7b2.ngbuilds.io/.

@mary-poppins
Copy link

You can preview 75b7911 at https://pr30731-75b7911.ngbuilds.io/.

Copy link
Member

@Splaktar Splaktar left a comment

Choose a reason for hiding this comment

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

LGTM

aio/content/guide/glossary.md Outdated Show resolved Hide resolved
@mary-poppins
Copy link

You can preview 8a85db1 at https://pr30731-8a85db1.ngbuilds.io/.

aio/content/guide/glossary.md Outdated Show resolved Hide resolved
packages/platform-server/PACKAGE.md Show resolved Hide resolved
Support for the various Angular run-time platforms is provided by the `@angular/platform-*` packages.
A platform can [polyfill](#polyfill) certain APIs or provide new platform-specific APIs.

* When running in a web browser, the [`platform-browser`](api/platform-browser) package supplies services that allow applications to access browser-specific features, such as the title of the document being displayed. All applications running in the browser use the same platform service.
Copy link
Contributor

Choose a reason for hiding this comment

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

platform- packages primarily enable apps written against @angular/core, @angualar/common and friends to execute in different environments by providing implementation for gathering user input and rendering UIs for the given platform. This allows Angular apps to be written In a platform independent way. The title service is one example of that but that API is quite insignificant compared to general dom manipulation apis that platforms abstract away.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@IgorMinar Please check rewrite

@jbogarthyde jbogarthyde requested a review from a team as a code owner June 5, 2019 15:08
@mary-poppins
Copy link

You can preview 0e13f48 at https://pr30731-0e13f48.ngbuilds.io/.

@mary-poppins
Copy link

You can preview a1affd6 at https://pr30731-a1affd6.ngbuilds.io/.

@jbogarthyde jbogarthyde force-pushed the jb-platform branch 2 times, most recently from ec1392b to b74ef59 Compare June 5, 2019 15:22
@mary-poppins
Copy link

You can preview ec1392b at https://pr30731-ec1392b.ngbuilds.io/.

@mary-poppins
Copy link

You can preview b74ef59 at https://pr30731-b74ef59.ngbuilds.io/.

aio/content/guide/glossary.md Outdated Show resolved Hide resolved
@@ -622,6 +622,19 @@ A class which is preceded by the `@Pipe{}` decorator and which defines a functio

To learn more, see [Pipes](guide/pipes).

{@a platform}

## platform
Copy link
Contributor

Choose a reason for hiding this comment

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

The entire section doesn't mention about platform APIs like PlatformRef and createPlatformFactory, but only the platform package.

But without mentioning platform APIs I'll ask what's the real difference between a platform package and non-platform package? If it's just about environment setup, will something like ng-jest being recognized a platform package?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is not platform doc, just a glossary definition that is supposed to clarify how we use the term "platform" in the context of Angular development. We probably do need better API and usage doc - we could open an issue for that.

@IgorMinar what do you think?

Copy link
Member

Choose a reason for hiding this comment

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

Initially I thought that this wasn't the place for those APIs mentioned by @trotyl, but after looking at the SSR guide and the platform-server/browser page (where they aren't mentioned) and reviewing how central they are to the platform packages, this might actually be a good place to link to and mention them briefly.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not much point in linking to them if they aren't discussed there. This should be mentioned in the issue, to make sure the link gets in when those docs are updated.

Copy link
Contributor

Choose a reason for hiding this comment

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

According to current concept, are platform-browser and platform-browser-dynamic same platform or not?

aio/content/guide/glossary.md Outdated Show resolved Hide resolved
aio/content/guide/glossary.md Outdated Show resolved Hide resolved
@mary-poppins
Copy link

You can preview d2e5bbe at https://pr30731-d2e5bbe.ngbuilds.io/.

@mary-poppins
Copy link

You can preview ee2a7dc at https://pr30731-ee2a7dc.ngbuilds.io/.

@mary-poppins
Copy link

You can preview 20eabfe at https://pr30731-20eabfe.ngbuilds.io/.

@jbogarthyde jbogarthyde force-pushed the jb-platform branch 2 times, most recently from bed5923 to 320d988 Compare June 10, 2019 16:36
@mary-poppins
Copy link

You can preview bed5923 at https://pr30731-bed5923.ngbuilds.io/.

@mary-poppins
Copy link

You can preview 320d988 at https://pr30731-320d988.ngbuilds.io/.

@mary-poppins
Copy link

You can preview 64072bf at https://pr30731-64072bf.ngbuilds.io/.

Copy link
Contributor

@IgorMinar IgorMinar left a comment

Choose a reason for hiding this comment

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

This looks good to me.

@IgorMinar IgorMinar added the action: merge The PR is ready for merge by the caretaker label Jun 12, 2019
AndrewKushnir pushed a commit that referenced this pull request Jun 12, 2019
@jbogarthyde jbogarthyde deleted the jb-platform branch June 13, 2019 16:13
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes effort1: hours freq2: medium risk: low target: patch This PR is targeted for the next patch release type: bug/fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants