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

3377 social icons #282

Open
wants to merge 35 commits into
base: develop
Choose a base branch
from
Open

3377 social icons #282

wants to merge 35 commits into from

Conversation

jarekzet
Copy link
Contributor

closes #3377

image

danielkryska and others added 30 commits October 25, 2021 12:13
RWD breakpoints update, contribution guide
[Fix] Missing secure attribute to cookies.
* feat(rendering): render elements by css class

* feat(render): inject components and rendering function into window

* Readme update and redering tools

* chore(readme): update readme badges

* fix(rendering): simplify rendering methods

* fix(rendering): fix rendering with css class and css id

* chore: update ue-information 

Co-authored-by: Pawel Gorczyca <goreck888@gmail.com>
* feat(rendering): render elements by css class

* feat(render): inject components and rendering function into window

* Readme update and redering tools

* chore(readme): update readme badges

* fix(rendering): simplify rendering methods

* fix(rendering): fix rendering with css class and css id

* Update ue-information (#184)

Remove the "EOSC Enhance" mention in the footer

* feat: hide auth button in main header on missing params

Co-authored-by: Pawel Gorczyca <goreck888@gmail.com>
chore: update catalog, and marketplace urls
feat: change link to catalogue, remove mp link
feat: restore catalogue & marketplace in header
feat: [#208] add monitoring and status links
feat: footer social logos, hover styles, copyright
New Header and footer links rollback
Update browse marketplace link
Release customizable terms of use
Release common profile links
# Conflicts:
#	src/main-header/main-header.interface.jsx
#	styles/main-header.scss
Release hide checkbox in header
Add maintenance text to the header
Release new design and links
Copy link

Hi @jarekzet.
Thank you for raising your pull request.
Please make sure you have followed our contributing guidelines. We will review it as soon as possible.

See your changes with

Documentation for development build

Development build

<script src="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.development.min.js"></script>
<link rel="stylesheet" href="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.development.min.css" />

Beta build

<script src="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.beta.min.js"></script>
<link rel="stylesheet" href="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.beta.min.css" />

Production build

<script src="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.production.min.js"></script>
<link rel="stylesheet" href="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.production.min.css" />

Run commons locally with production build

  • copy paste beneath snippet to index.html
  • run index.html in a browser
<!DOCTYPE html>
<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
  </head>
  <body>
    <eosc-common-main-header
      username="name surname"
      login-url="https://marketplace.eosc-portal.eu/users/auth/checkin"
      logout-url="https://marketplace.eosc-portal.eu/users/logout"
    ></eosc-common-main-header>

    <EoscCommonMainFooter></EoscCommonMainFooter>
    <script src="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.production.min.js"></script>
    <link rel="stylesheet" href="https://s3.cloud.cyfronet.pl/eosc-portal-common/pr-3377-social-icons/index.production.min.css" />
  </body>
</html>

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.

6 participants