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

EPHEH-313: Color Schema v2. #581

Merged
merged 14 commits into from
Mar 15, 2024
Merged

EPHEH-313: Color Schema v2. #581

merged 14 commits into from
Mar 15, 2024

Conversation

tibi2303
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Feb 28, 2024

🚀 Deployed on https://preview-581--oelibrary.netlify.app

@github-actions github-actions bot temporarily deployed to pull request February 28, 2024 22:16 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 29, 2024 00:14 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 29, 2024 00:27 Inactive
@github-actions github-actions bot temporarily deployed to pull request February 29, 2024 00:34 Inactive
*
* Example config object: {
*
* color-schema: [
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is already nice but for the final version I would like a proper docs page, where the purpose, prerequisites, usage are explained, including the possibility to use directly in a central sass file or through the builder, how the builder.config can be set up etc
Also words about what it offers, example usage of the classes, and about the mixins per components, how you can use them

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I wrote the documentation. Tell me please if it sounds clear:
https://github.com/openeuropa/bootstrap-component-library/tree/EPHEH-313-V2/builder

I added in the builder as always we had for all the scripts used.

src/themes/default/src/scss/color-schema.scss Outdated Show resolved Hide resolved
src/themes/default/src/scss/color-schema.scss Outdated Show resolved Hide resolved
src/compositions/bcl-color-schema/color-schema.story.js Outdated Show resolved Hide resolved
src/compositions/bcl-color-schema/color-schema.story.js Outdated Show resolved Hide resolved
src/compositions/bcl-color-schema/package.json Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to pull request March 1, 2024 13:29 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 5, 2024 16:46 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 5, 2024 17:19 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 6, 2024 09:29 Inactive
```javascript
color-scheme: [
{
entry (string - source file of color-schema),
Copy link
Collaborator

Choose a reason for hiding this comment

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

its a bit unclear, maybe something more concrete but descriptive

Suggested change
entry (string - source file of color-schema),
entry ../path/to/you-colour-scheme.scss,

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done.

```javascript
color-scheme: [
{
entry (string - source file of color-schema),
Copy link
Collaborator

Choose a reason for hiding this comment

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

rename all instances of schema to scheme

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

renamed.


The 'link' variable changes the color of all the links inside the schema.

All of these are possible through a namespace class `.{$color-schema-name}`. For example, a banner which uses the color schema will have class `'bcl-banner pixy-pink'`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

to much info, confusing

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

removed it.

Comment on lines 109 to 118
The color-scheme script makes use of these files like this:

```javascript
const scssContent = imports + "\n" + entryVariables + "\n" + generator;
```

- Imports: The necessary imports from Bootstrap and also the mixins.
- EntryVariables: The Color schema added by the user (source file).
- Generator: The call of the mixins used.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think again to much info

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

removed it.

Comment on lines +10 to +17
* color-scheme: [
* {
* entry: [path.resolve(nodeModules, "myfile.js")],
* dest: path.resolve(outputFolder, "js"),
* options: {
* includePaths,
* sourceMap: "file",
* },
Copy link
Collaborator

Choose a reason for hiding this comment

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

yes, something like this in the readme

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

added also in readme.

@github-actions github-actions bot temporarily deployed to pull request March 13, 2024 12:41 Inactive
@github-actions github-actions bot temporarily deployed to pull request March 13, 2024 13:20 Inactive
@tibi2303 tibi2303 merged commit e06b4c6 into EPHEH-313 Mar 15, 2024
7 of 9 checks passed
@tibi2303 tibi2303 deleted the EPHEH-313-V2 branch March 15, 2024 09:43
drishu pushed a commit that referenced this pull request Aug 1, 2024
* EPHEH-313: Color Schema v2.

* EPHEH-313-V2: Fix for accordion.

* EPHEH-313-V2: Fix for css.

* EPHEH-313-V2: Fix for css.

* EPHEH-313-v2: PR feedback.

* EPHEH-313-v2: Changed color schema.

* EPHEH-313: Color schema renaming to Color Scheme.

* EPHEH-313: Improved.

* EPHEH-313: Improved documentation.

* EPHEH-313: Improved documentation.

* EPHEH-313: Updated doc.

* EPHEH-313: Updated doc.

* EPHEH-313: Updated doc.

* EPHEH-313: Updated doc.

---------

Co-authored-by: Tiberiu Dumitru <[email protected]>
tibi2303 added a commit that referenced this pull request Aug 1, 2024
* EPHEH-313: Added yml-importer package.

* Refactor approach.

* Changed location of importer.

* EPHEH-313: V1 color schema.

* EPHEH-313: Removed base.

* EPHEH-313: Color Schema v2. (#581)

* EPHEH-313: Color Schema v2.

* EPHEH-313-V2: Fix for accordion.

* EPHEH-313-V2: Fix for css.

* EPHEH-313-V2: Fix for css.

* EPHEH-313-v2: PR feedback.

* EPHEH-313-v2: Changed color schema.

* EPHEH-313: Color schema renaming to Color Scheme.

* EPHEH-313: Improved.

* EPHEH-313: Improved documentation.

* EPHEH-313: Improved documentation.

* EPHEH-313: Updated doc.

* EPHEH-313: Updated doc.

* EPHEH-313: Updated doc.

* EPHEH-313: Updated doc.

---------

Co-authored-by: Tiberiu Dumitru <[email protected]>

* EPHEH-313: Updated readme.

* EPHEH-313: Updated readme.

* EPHEH-313: Feedback.

* EPHEH-313: Fix for utilities classes.

* EPHEH-313: Added facts icon to primary color and updated default color-scheme.

* EPHEH-313: Improved classname.

* EPHEH-313: Removed duplicated packages.

---------

Co-authored-by: Tiberiu Dumitru <[email protected]>
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.

2 participants