Skip to content
This repository was archived by the owner on Oct 13, 2021. It is now read-only.

Conversation

@ericpoirier
Copy link
Contributor

Change-Id: I588a135554529922f2599869fca5fcd166d173e9
Signed-off-by: Eric Poirier eric.poirier@eclipse-foundation.org

@netlify
Copy link

netlify bot commented Nov 19, 2020

Deploy preview for eclipsefdn-solstice-assets ready!

Built with commit e860feb

https://deploy-preview-197--eclipsefdn-solstice-assets.netlify.app

@ericpoirier ericpoirier marked this pull request as draft November 19, 2020 15:08
Copy link
Contributor

@flora8984461 flora8984461 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 the folder path is not corresponding, one is under less/_components/quicksilver/ while the import is ../_components/main-sidebar.less

@autumnfound
Copy link

Are we still using this PR?

@chrisguindon
Copy link
Member

@ericpoirier can we close this?

@chrisguindon chrisguindon reopened this Dec 3, 2020
@chrisguindon
Copy link
Member

@ericpoirier can we close this?

Eric, can we close this? If not, can we finish this?

@ericpoirier
Copy link
Contributor Author

@ericpoirier can we close this?

Eric, can we close this? If not, can we finish this?

I'll rebase and see if there is more work to be done to close this.

@ericpoirier ericpoirier force-pushed the epoirier/master/add_main_sidebar branch from afeddb3 to 7c85f5c Compare January 19, 2021 18:29
@ericpoirier ericpoirier force-pushed the epoirier/master/add_main_sidebar branch from 7c85f5c to c602c6e Compare January 20, 2021 16:27
@ericpoirier ericpoirier force-pushed the epoirier/master/add_main_sidebar branch from c602c6e to 07e6691 Compare January 21, 2021 13:47
@chrisguindon
Copy link
Member

As a side note, I started to use this new sidebar on newer sites:
https://eclipsefdn-osgi.netlify.app/about/faq/
https://eclipsefdn-asciidoc.netlify.app/about/

..whatever solution we take, we must make sure not to break older sites with the previous sidebar such as https://jakarta.ee/about/

Change-Id: I588a135554529922f2599869fca5fcd166d173e9
Signed-off-by: Eric Poirier <eric.poirier@eclipse-foundation.org>
Change-Id: Id7a590db2b8c80d0dafd8317da52929cf41b6745
Signed-off-by: Eric Poirier <eric.poirier@eclipse-foundation.org>
@ericpoirier ericpoirier force-pushed the epoirier/master/add_main_sidebar branch from 07e6691 to cde6bdc Compare February 25, 2021 19:45
@ericpoirier ericpoirier marked this pull request as ready for review February 25, 2021 19:46
Change-Id: I51c448156ec6ce8030a71dc369f063e72efc8c33
Signed-off-by: Eric Poirier <eric.poirier@eclipse-foundation.org>
@chrisguindon chrisguindon merged commit 47014c3 into EclipseFdn:master Feb 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants