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.

Sign up for GitHub

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

Jump to bottom

Add user references and logos #857

Merged
merged 1 commit into from
Jul 9, 2019
Merged

Add user references and logos #857

merged 1 commit into from
Jul 9, 2019

Conversation

dhmlau
Copy link
Member

@dhmlau dhmlau commented Jul 8, 2019

Updating the LB web site with our users (logos + testimonials). See details in loopbackio/loopback-next#3047. Got the design from the designer (thanks Mihir!)

  • The order of the logos is random.
  • I'm trying to make the logos more or less the same size, so the width value is not the same. If anyone has better ways, please let me know!

There are 2 sections added in this PR.

  1. "who's using LoopBack"

screencap1

  1. "what our users say"

screencap2

second half of the "what our users say"
screencap3

@dhmlau dhmlau requested a review from bschrammIBM as a code owner July 8, 2019 19:24
@dhmlau
Copy link
Member Author

dhmlau commented Jul 8, 2019

cc @marioestradarosa @vvdwivedi @samarpanB @kowsheek

@dhmlau dhmlau self-assigned this Jul 8, 2019
raymondfeng
raymondfeng approved these changes Jul 9, 2019
View reviewed changes
Copy link
Contributor

@raymondfeng raymondfeng left a comment

Choose a reason for hiding this comment

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

LGTM with my local preview.

One minor thing - do we want to make the logos clickable to their site?

@dhmlau
Copy link
Member Author

dhmlau commented Jul 9, 2019

One minor thing - do we want to make the logos clickable to their site?

That's good point. Let me do it in a separate PR. Thanks!

@dhmlau dhmlau merged commit f5e0d61 into gh-pages Jul 9, 2019
@delete-merged-branch delete-merged-branch bot deleted the users branch July 9, 2019 17:21
@dhmlau dhmlau mentioned this pull request Jul 9, 2019
Merged
@agnes512 agnes512 mentioned this pull request Aug 6, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jannyHou jannyHou jannyHou approved these changes

@raymondfeng raymondfeng raymondfeng approved these changes

@bschrammIBM bschrammIBM Awaiting requested review from bschrammIBM

Assignees

@dhmlau dhmlau

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dhmlau @raymondfeng @jannyHou