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

Update license to v1.2 #1711

Merged
merged 25 commits into from
Oct 12, 2022
Merged

Update license to v1.2 #1711

merged 25 commits into from
Oct 12, 2022

Conversation

aurindam
Copy link
Member

@aurindam aurindam commented Oct 6, 2022

  1. Update Slint License Agreement
  2. Update README.md (root)
  3. Update LICENSE.md (root)
  4. Update FAQ.md (root)

FAQ.md Outdated Show resolved Hide resolved
FAQ.md Outdated Show resolved Hide resolved
FAQ.md Outdated Show resolved Hide resolved
LICENSES/LicenseRef-Slint-commercial.md Outdated Show resolved Hide resolved
LICENSES/LicenseRef-Slint-commercial.md Outdated Show resolved Hide resolved
LICENSES/LicenseRef-Slint-commercial.md Outdated Show resolved Hide resolved
LICENSES/LicenseRef-Slint-commercial.md Outdated Show resolved Hide resolved
LICENSES/LicenseRef-Slint-commercial.md Outdated Show resolved Hide resolved
LICENSES/LicenseRef-Slint-commercial.md Show resolved Hide resolved
@hunger
Copy link
Member

hunger commented Oct 6, 2022

I only added formal/formatting things... feel free to ignore:-)

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Member

@ogoffart ogoffart left a comment

Choose a reason for hiding this comment

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

Just one comment in the FAQ.

FAQ.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
FAQ.md Outdated Show resolved Hide resolved
FAQ.md Show resolved Hide resolved
FAQ.md Outdated Show resolved Hide resolved
FAQ.md Outdated Show resolved Hide resolved
FAQ.md Outdated Show resolved Hide resolved
LICENSES/LicenseRef-Slint-commercial.md Outdated Show resolved Hide resolved
README.md Outdated
@@ -8,46 +8,49 @@
[![REUSE status](https://api.reuse.software/badge/github.com/slint-ui/slint)](https://api.reuse.software/info/github.com/slint-ui/slint)
[![Discussions](https://img.shields.io/github/discussions/slint-ui/slint)](https://github.com/slint-ui/slint/discussions)

Slint is a toolkit to efficiently develop fluid graphical user interfaces for any display: embedded devices and desktop applications. We support multiple programming languages, such as
Rust, C++, and JavaScript.
Slint is a toolkit to efficiently design, develop and deploy responsive apps on any screen. Frontends built with Slint can be easily integrated with backends written in different programming languages such as C/C++, Rust, and JavaScript. Slint is developed by SixtyFPS GmbH together with the community. We invite you to use Slint and be part of its community.
Copy link
Member

Choose a reason for hiding this comment

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

Interesting introduction of frontend and backend here.

Copy link
Member Author

@aurindam aurindam Oct 10, 2022

Choose a reason for hiding this comment

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

Open to changes in the wording :)

Copy link
Member

Choose a reason for hiding this comment

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

Re-reading this, I think the previously sentence was simpler and easier to read. Introducing terms such as "backend" and "frontend" seems tempting from a technical perspective, but it also creates ambiguity for people coming from the web for example. Does this mean that the backend runs on a server and Slint UI runs elsewhere? (traditionally there's a host boundary between those).

Since this is the very first sentence of the README, simplicity and clarify wins over the "feature" that we support a frontend and backend separation. If anything, I think that could be a bullet point under "features" somewhere.

I suggest to go back to the old wording.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe Design vs. Logic?

Suggested change
Slint is a toolkit to efficiently design, develop and deploy responsive apps on any screen. Frontends built with Slint can be easily integrated with backends written in different programming languages such as C/C++, Rust, and JavaScript. Slint is developed by SixtyFPS GmbH together with the community. We invite you to use Slint and be part of its community.
Slint is a toolkit to efficiently design, develop and deploy responsive apps on any screen. User interface design built with Slint can be easily integrated with logic written in different programming languages such as C++, Rust, and JavaScript. Slint is developed by SixtyFPS GmbH together with the community. We invite you to use Slint and be part of its community.

I'm also unsure if we should mention SixtyFPS GmbH so early in the introduction.

Copy link
Member Author

Choose a reason for hiding this comment

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

reverted back to old wording. Added one sentence "We invite you to use Slint and be part of its community."

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Copy link
Member

@ogoffart ogoffart left a comment

Choose a reason for hiding this comment

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

I think it's good now
(please squash the commit while merging)

README.md Outdated Show resolved Hide resolved
@aurindam aurindam merged commit 5d06db4 into master Oct 12, 2022
@aurindam aurindam deleted the update-license branch October 12, 2022 12:53
tronical pushed a commit that referenced this pull request Oct 12, 2022
Update to v1.2 and updated the README and FAQ.
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.

4 participants