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

Content improvements #354

Merged
merged 66 commits into from Sep 7, 2023
Merged

Content improvements #354

merged 66 commits into from Sep 7, 2023

Conversation

kaishin
Copy link
Collaborator

@kaishin kaishin commented Aug 15, 2023

This PR aims to introduce the content changes that the SWWG have worked on since last year.
Several changes have been made to the content of the most visited pages of the website, including the landing page, the getting started page, and the install page.

The main changes are:

  • New content added to the landing page to give more guidance to visitors via clear navigation buttons, code snippets, and two new "Use cases" and "Getting involved" sections.
  • The "Getting started" page has been reworked to provide a more structured and guided approach to using Swift. It now features links to four brand new guides to get started, with additional resources for seasoned users in the "Go further" section.
  • The "Install" page has been streamlined for more clarity on how to install Swift on various platforms. Each platform now has its own dedicated page.

The PR can be previewed here.

Checklist

The following are steps that need to be taken before merging this PR.

  • Update code snippets (@alexandersandberg)
  • Update redhat download link for 5.8.1 (@shahmishal)
  • Publish guide source code examples
  • Get up-to-date with latest main
  • Fix SVG colors in landing page when themes are explicitly set. (Comment) (@kaishin)

kaishin and others added 30 commits September 13, 2022 00:44
Co-authored-by: Alexander Sandberg <hi@alexandersandberg.com>
* Adding Getting Started tutorial for SwiftUI / iOS 16.

* Corrections from James and Reda.

* Update getting-started/swiftui/index.md

Co-authored-by: Alexander Sandberg <hi@alexandersandberg.com>
* Add data file with items for "Go Further" section

* Make thumbnail-url values null with a TBD comment

* Add missing sentence from Swift strings entry
motivation: getting started guides

changes:
* add a cli guide
* add a library guide + link from index page
* share install instructions across pages
* Tweaked the very start of the new Getting Started page.

* Added a link to TSPL in case people want to get started with that type of resource.

* Linked to the package manager page.

* Introduced SwiftPM on the package manager page, too.
* add documentation section to getting started

* remove absolute path from go further resource url

* merge documentation section into other sections

* update sidebar navigation

* rewrite tspl paragraph
* Add Vapor Getting Started guide

* Add link to Codable

* Update getting-started/vapor-web-server/index.md

Co-authored-by: Max Desiatov <m_desiatov@apple.com>

* Code review comments

---------

Co-authored-by: Max Desiatov <m_desiatov@apple.com>
motivation: address feedback from content review

changes:
* slight langauge changes
* update vapor guide to be more generic so that we an also point to other web frameworks in the future


Co-authored-by: Tim Condon <0xTim@users.noreply.github.com>
* Format Swift code correctly

* Add images

* Add link to tutorial

* Update images and use video instead of gif

---------

Co-authored-by: Reda Lemeden <git@redalemeden.com>
…with item for Structures and Classes chapter in TSPL (#254)
Co-authored-by: Alexander Sandberg <hi@alexandersandberg.com>
* fix letter casing on landing page

* update latest release to 5.8 on landing page
- Use latest URL for Swift tour in TSPL
- Reorder items
- Add title and blurb for Value and Reference Types blog post
motivation: address feedback from core-team

changes:
* improve description of landing page use cases
* improve description of getting started guides
* add "cross platform command-line tool" as a landing page use case, link to getting started guides (TODO: come up with better)

Co-authored-by: Alexander Sandberg <hi@alexandersandberg.com>
* fix latest release button smaller than rest when column

* WIP: add featured snippets to landing page

* ensure that snippet visibility rules are not overridden

* formatting

* remove important from css rules

* add height to featured snippet to prevent layout shifts

* move snippet to top of landing page

* remove long snippet example
* New “Swift is…” copy.

* Tweaked preamble wording.

* Removed bolding from the About page.

* Typo.

* Whitespace.

* Fixed up sentence in the About page.

* More minor wording tweaks.

* Removed extra space.

* Suggestion from Reda on the quick and fast line.
index.md Outdated Show resolved Hide resolved
index.md Outdated Show resolved Hide resolved
index.md Outdated Show resolved Hide resolved
index.md Outdated Show resolved Hide resolved
index.md Outdated Show resolved Hide resolved
about/index.md Outdated Show resolved Hide resolved
shahmishal and others added 2 commits September 6, 2023 17:32
Co-authored-by: tomer doron <tomer@apple.com>
Co-authored-by: tomer doron <tomer@apple.com>
@tkremenek tkremenek self-requested a review September 7, 2023 13:46
Copy link
Contributor

@stevapple stevapple left a comment

Choose a reason for hiding this comment

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

Still some questions on the Windows installation part.

#### Installation Instructions

##### Install using the Windows Package Manager
## Install using the Windows Package Manager

The [Windows Package Manager](https://docs.microsoft.com/windows/package-manager/) can be found in the [Microsoft Store](https://www.microsoft.com/en-us/p/app-installer/9nblggh4nns1) or be [installed directly](ms-appinstaller:?source=https://aka.ms/getwinget).
Copy link
Contributor

Choose a reason for hiding this comment

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

Something we missed from the prior PR: WinGet is preinstalled on Windows 11 (21H2 and later) now.

Comment on lines -126 to -129
* Using Scoop:
~~~ cmd
scoop install swift
~~~
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this omitted?


1. Install Swift:
0. Install Swift:

Swift can be installed through the official installer directly, or using the Windows Package Manager as well. Notice that Windows Package Manager release may be behind the official release.
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
Swift can be installed through the official installer directly, or using the Windows Package Manager as well. Notice that Windows Package Manager release may be behind the official release.
Swift can be installed through the official installer directly, or using supported package managers as well. Notice that latest release from the supported package managers may lag behind the official release.

@@ -110,8 +55,8 @@ Invoke-RestMethod -Url 'get.scoop.sh' | Invoke-Expression
~~~ cmd
pip install six
~~~

1. Install Swift:

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

Comment on lines -64 to -67
#### Enabling Developer Mode

In order to develop applications, particularly with the Swift Package Manager, you will need to enable developer mode. Please see Microsoft's [documentation](https://docs.microsoft.com/en-us/windows/apps/get-started/enable-your-device-for-development) for instructions for enabling developer mode.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this dropped? Developer mode is still required for SwiftPM. It could be mentioned later, but definitely worth the attention.

#### Installation Instructions

##### Install using the Windows Package Manager
## Install using the Windows Package Manager
Copy link
Contributor

Choose a reason for hiding this comment

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

The title should be updated to indicate both WinGet and Scoop. Perhaps a general “Install using package manager”?

In fact I think it may be worth splitting these two because now they’re mostly independent.

Copy link
Member

Choose a reason for hiding this comment

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

@stevapple Would you be able to create a PR with suggested changes?

Copy link
Contributor

Choose a reason for hiding this comment

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

shahmishal and others added 2 commits September 7, 2023 08:41
Tweak language on how Swift is good as a general-purpose programming language, focusing on "joy" instead of "quick to write."

Co-authored-by: tomer doron <tomer@apple.com>
@kaishin kaishin assigned kaishin and unassigned 0xTim Sep 7, 2023
@kaishin
Copy link
Collaborator Author

kaishin commented Sep 7, 2023

@swift-ci test

@kaishin kaishin merged commit b5e0e4c into main Sep 7, 2023
1 check passed
@kaishin kaishin deleted the content-improvements branch September 7, 2023 18:06
@kaishin
Copy link
Collaborator Author

kaishin commented Sep 7, 2023

🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet