Skip to content

create new css layout 'resume-simple'#826

Open
Willian1661 wants to merge 2 commits intoshield-wall:masterfrom
Willian1661:resume-simple-layout
Open

create new css layout 'resume-simple'#826
Willian1661 wants to merge 2 commits intoshield-wall:masterfrom
Willian1661:resume-simple-layout

Conversation

@Willian1661
Copy link
Contributor

No description provided.

@eerison eerison self-requested a review April 29, 2025 09:57
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
<title>Myprofile resume</title>
<script type="module" src="src/main.js"></script>
<link rel="stylesheet" href="src/style/resume-default/main.scss">
Copy link
Collaborator

Choose a reason for hiding this comment

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

revert this change.

@@ -0,0 +1,212 @@
@import "bulma/bulma.sass";
Copy link
Collaborator

Choose a reason for hiding this comment

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

not needed!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i wanted to use the default style of bulma for the small pieces, paragraphs ; font size, font style; all the small pieces.

@@ -0,0 +1,212 @@
@import "bulma/bulma.sass";
@import "../icons";
Copy link
Collaborator

Choose a reason for hiding this comment

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

do we need icons in for simple resume?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

no if we're following this structure example, at first ,i didn't mean to make it completely identical to this one:

Screenshot from 2025-04-30 11-16-44

border-bottom: 1px solid $primary-light-color;
}

.has-tag {
Copy link
Collaborator

Choose a reason for hiding this comment

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

we do not need tags!

}
}

.has-tags {
Copy link
Collaborator

Choose a reason for hiding this comment

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

it is duplicated! but probably you can remove it :)

}
}

.myprofile-block:not(.basic):not(.has-tag):not(.has-tags):not(.has-list):not(.has-timeline) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

do you need this css? and the whole css bellow?

@eerison eerison marked this pull request as draft April 30, 2025 10:33
@eerison
Copy link
Collaborator

eerison commented Apr 30, 2025

Hey @Willian1661

i converted this PR to draft, after you finish you put it ready to review again!

@eerison eerison mentioned this pull request Apr 30, 2025
5 tasks
@Willian1661 Willian1661 force-pushed the resume-simple-layout branch from 0ace266 to 3b7a8bc Compare May 6, 2025 21:30
@Willian1661 Willian1661 force-pushed the resume-simple-layout branch from 06e02b5 to 19cf3b5 Compare May 7, 2025 20:40
@Willian1661 Willian1661 marked this pull request as ready for review May 7, 2025 20:41
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