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

first draft personal portfolio #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
50 changes: 47 additions & 3 deletions pages/about.html
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,53 @@
<meta charset="UTF-8">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Document</title>
<link href="../styles/styles.css" rel="stylesheet">
<link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
<link href="https://fonts.googleapis.com/css2?family=Fira+Sans:wght@300&display=swap" rel="stylesheet">
<title>About Tyrah</title>
</head>
<body>

<body class="about-bg">

<nav>
<header>
<h1>☁️Welcome to TyrahForm 🌬️</h1>
<div class="flex-container">
<a href="index.html">Home</a>
<a href="portfolio.html">View My Portfolio</a>
</div>
</header>
</nav>

<!-- 🫧 -->
<div class="about-photos">

Choose a reason for hiding this comment

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

Consider wrapping all this content in a main tag to set off the main content of the page. There was a project requirement to use header, main, and footer on your pages.

<div class="div-photo-1"><img src="../photos/rainbow.jpg" alt="tyrah standing at a vending maching" height="300" width="300" class="about-photos"></div>
</div>
<block>

Choose a reason for hiding this comment

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

block is not a valid HTML tag.

<p>Hi, 🫧 I'm Tyrah and I'm in Cohort 17 of the Ada Developer's Academy code bootcamp for women and gender expansive folx. I'm learning
full stack web development with Python, Flask, PSQL and React Js.</p>
<p>Before Ada, I was a cybersecurity student studying network engineering with intent to get my certification and I plan to go back to that course of study.
My studies of internet systems and architecture has led to an interest in cloud technologies and I would love to develop my skills and gain certifications in this area too.
I see cloud computing as an economical way to grow computer systems and facilitate collaboration to develop user-friendly softwares and better the lives of people in my community.
</p>
<div class="about-photos">
<div class="div-photo"><img src="../photos/vending-machine.jpg" alt="tyrah standing at a vending maching" height="300" width="300" class="about-photos"></div>
</div>
Comment on lines +36 to +38

Choose a reason for hiding this comment

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

Are these nested divs required? Could the styles being applied to the div wrappers go directly on the images?

<p>
I found footing in studying tech via clubmanship. I was a member of Society for Advancement of Chicanos/Hispanics & Native Americans in Science and formed a chapter of Women in Science and Engineering
at Seattle Central College with the help of the presidents of SACNAS and worked in close collaboration with them on projects and fundraisers for two years including work on a video game with the Unity
game engine which earned placement at a student tech convention in Hawaii.
</p>
<div class="about-photos">
<div class="div-photo-2"><img src="../photos/graywallblackhat.jpg" alt="tyrah standing at a vending maching" height="300" width="300" class="about-photos"></div>
</div>
<p>
I'm happy Ada and my tech clubs have given me a space to learn and grow as a black woman searching for her place in the tech industry and I hope to continue this sense of community into my career by contributing to technologies
that encourage community, innovative ways to network and platforms to give the marginalized a voice such as social media.
</p>
</block>
<div class="fixed-footer">
<div class="container"><h4>Copyright &copy; 2022 The Pink Hat Hacker</h4></div>
</div>
</body>
</html>
Empty file added pages/blog.html
Empty file.
43 changes: 40 additions & 3 deletions pages/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,46 @@
<meta charset="UTF-8">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Document</title>
<link href="../styles/styles.css" rel="stylesheet">
<link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
<link href="https://fonts.googleapis.com/css2?family=Fira+Sans:wght@300&display=swap" rel="stylesheet">
Comment on lines +8 to +10

Choose a reason for hiding this comment

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

Consider using the @import directive style of font importing, which can be placed in a css file rather than the html.

<title>Tyrah Gullette</title>
</head>
<body>

<body class="index-background">
<nav>
<header>
Comment on lines +14 to +15

Choose a reason for hiding this comment

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

Consider reversing these tags so that header contains nav. Currently, this implies the contents are the header of the nav, rather than being the header of the page which happens to contain a nav.

<h1>Tyrah Gullette</h1>

Choose a reason for hiding this comment

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

You have 2 h1s on the page, which cause a validation warning. It's recommended to have only a single h1 on the page.

<div class="flex-container">
<a href="portfolio.html">My Portfolio</a>
<a href="about.html">About Me</a>
</div>
</header>
</nav>

<div>

Choose a reason for hiding this comment

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

Consider using a main tag to indicate where the main content of the page is. There was a project requirement to use header, main, and footer on your pages.

<div><h1>Tyrah Gullette</h1></div>

Choose a reason for hiding this comment

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

Here's the other h1, which for me at least ends up behind the nav header. Consider removing this, and using styling to push down the content to be below the header area.

<div><h2>Full Stack Web Developer | Cloud Curious | Cybersecurity | Network Engineering</h2></div>
</div>

<div class="picture-container">
<img src="../photos/purple-bg.jpg" alt ="abstract purple background" height="400" width="500" class="picture">
<p class="text">Do you like cryptography?</p>
<p class="overlay overlay-text">I can fly but I have no wings. I can cry but I have no eyes. Wherever I go, darkness follows me. What am I?</p>
</div>


<div class="flex-descriptions">
<div><h3>Curiosity</h3></div>
<div><h3>Leadership</h3></div>
<div><h3>Organization</h3></div>
<div><h3>User-Friendly</h3></div>
<div><h3>Detailed</h3></div>
<div><h3>Supportive</h3></div>
</div>
Comment on lines +36 to +43

Choose a reason for hiding this comment

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

This feels more like a list of terms to me than a series of headings. We should avoid using heading tags solely for their styling connotations of being larger, which can be accomplished through style rules instead. We should think of headings as the content that would be pulled out into a document outline or table of contents for the page.

<div class="fixed-footer">

Choose a reason for hiding this comment

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

Consider using a footer tag rather than a custom class. There was a project requirement to use header, main, and footer on your pages.

<div class="container"><h4>Copyright &copy; 2022 The Pink Hat Hacker</h4></div>
</div>

</body>
</html>
30 changes: 27 additions & 3 deletions pages/portfolio.html
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,33 @@
<meta charset="UTF-8">
<meta http-equiv="X-UA-Compatible" content="IE=edge">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>Document</title>
<link href="../styles/styles.css" rel="stylesheet">
<link rel="preconnect" href="https://fonts.googleapis.com">
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
<link href="https://fonts.googleapis.com/css2?family=Fira+Sans:wght@300&display=swap" rel="stylesheet">
<title>Tyrah's Portfolio</title>
</head>
<body>

<body class="portfolio-background">
<nav>
<header>
<h1>Portfolio</h1>
<div class="flex-container">
<a href="index.html">Home</a>
<a href="about.html">About Me</a>
</div>
</header>
</nav>
<div class="grid-container">
<div class="project-1"><h2><a href="https://github.com/ada-c17/task-list-api">Task List</a></h2></div>
<div class="project-2"><h2><a href="https://github.com/ursaturnine/swap-meet">Swap Meet</a></h2></div>
<div class="project-3"><h2><a href="https://github.com/ursaturnine/tyrah-and-san-solar-system-api">Solar System Api</a></h2></div>
<div class="project-4"><h2><a href="https://github.com/ursaturnine/viewing-party">Viewing Party</a></h2></div>

</div>
Comment on lines +23 to +29

Choose a reason for hiding this comment

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

If you would like to use headings for each of the project names, consider using a section as the wrapping element rather than a div. But generally, heading tags should be considered as headings on the page (with associated content). To me, this feels like ti could be more semantically expressed as a list. Styling would be able to make it visually appear the same way.



<div class="fixed-footer">
<div class="container"><h4>Copyright &copy; 2022 The Pink Hat Hacker</h4></div>
</div>
</body>
</html>
Binary file added photos/fremontflower.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added photos/graywallblackhat.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added photos/purple-bg.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added photos/purplebham.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added photos/rainbow.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added photos/reddress.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added photos/sisters.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added photos/vending-machine.jpg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
212 changes: 212 additions & 0 deletions styles/styles.css
Original file line number Diff line number Diff line change
@@ -0,0 +1,212 @@
a {
color: #320c1b;
box-shadow: 5px 5px 5px #392c58;
flex-direction: column;
background-color: #c3a4db;
border: 1px solid;
border-radius: 25px;
padding: .5rem;
transition: all .5s ease-out;
}

a:hover,
a:focus {
background-color: #dcdadb;
color: #320c1b;
}

.about-photos {
border-radius: 200px;
margin: 5px;
}


.about-bg {
background-image: conic-gradient(from 90deg, #637cde, #b163de, #81afe4, #a1c0e2, #beb3f4);
border-radius: 50%;
text-align: center;
margin: 20px;
scrollbar-width: thin;
scrollbar-color: var(--thumbBG) var(--scrollbarBG);
padding-top: 60px;
padding-bottom: 40px;
}


block {
font-family: 'Fira Sans', sans-serif;
font-size: xx-large;
color:#392c58;
text-align: justify;
}

div > h2{
text-shadow: 2px 2px 5px #e3bce6;
text-align: center;
padding: 1em;
}


div > h3 {
padding: .5em 10em 50em 0em;
}

.div-photo {
float: right;
shape-outside: circle();

Choose a reason for hiding this comment

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

Cool way to make the surrounding text wrap to the rounded shape of the images.

margin: 10px;
}
.div-photo-1, .div-photo-2 {
float:left;
shape-outside: circle();
margin: 10px;
}

.flex-container {
display: flex;
justify-content: right;
color: #320c1b;
gap: 1rem;
padding: 7px 25px;
position: fixed;
top: 0px;
}


.flex-descriptions {
font-family: 'Fira Sans', sans-serif;
display: flex;
padding: 1em 3em 2em 5em;
gap: .1rem;
font-size: xx-large;
text-shadow: 2px 2px 5px #e3bce6;
flex-direction: column;
justify-content: space-between;
}

.grid-container {
align-items: center;
display: grid;
width: 30vw;
height: 80vh;
grid-template: 50% 50% / 600px 600px;
}


h1, h2, h3 {
font-family: 'Fira Sans', sans-serif;
font-size: xx-large;
font-weight: lighter;
color: silver;
}

h4 {
font-family: 'Fira Sans', sans-serif;
font-size: small;
font-weight: lighter;
color:#392c58;
}

h1 {
text-align: center;
text-shadow: 2px 2px 4px #af92f6;
}


header, .fixed-footer {
width: 100%;
position: fixed;
background-image: conic-gradient(from 90deg, #637cde, #b163de, #81afe4, #a1c0e2, #beb3f4);
padding: 1px 0;
}


header {
top: 0px;
}


.fixed-footer {
bottom: 0;
}

.index-background {
background-image: radial-gradient(circle, purple,pink, hotpink, magenta,lavender,indigo,violet);

Choose a reason for hiding this comment

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

Love all the cool gradients you used!

scrollbar-width: thin;
scrollbar-color: var(--thumbBG) var(--scrollbarBG);

Choose a reason for hiding this comment

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

I don't think this actually has any effect since these variables need to be assigned before they can be used.

https://developer.mozilla.org/en-US/docs/Web/CSS/--*

}

.overlay {
position: absolute;
border-radius: 5px;
bottom: 0;
left: 0;
right: 0;
background-color: purple;
overflow: hidden;
width: 100%;
height: 100%;
transform: scale(0);
transition: .3s ease;
}


.overlay-text {
color: #c3a4db;
font-size: 3px;
font-family: 'Fira Sans', sans-serif;
position: absolute;
top: 250px;
left: 1150px;
transform: translate(-50%, -50%);
text-align: center;
}


.picture {
position: absolute;
left: 950px;
top: 100px;
border-radius: 300px;
}


.picture-container {
height: 50px;
width: 50px;
position: relative;
}


.picture-container:hover .overlay {
transform: scale(10);
}

.portfolio-background {

Choose a reason for hiding this comment

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

Consider using the styles.css file as a place to put rules that are shared across all your pages, and then make individual css files for each page to add in the style rules specific to that page.

background-image: radial-gradient(circle, purple,pink, hotpink, magenta,lavender,indigo,violet);
scrollbar-width: thin;
scrollbar-color: var(--thumbBG) var(--scrollbarBG);
padding-top: 60px;
padding-bottom: 40px;
}

.project-1 h2 a, .project-2 h2 a, .project-3 h2 a, .project-4 h2 a{
border-radius: 150px;
border: solid white;
padding: 80px;
box-shadow: 2px 7px #dcdadb;
}



.text {
z-index: 100;
position: absolute;
color: white;
font-family: 'Fira Sans', sans-serif;
font-size: 60px;
font-weight: bold;
left: 1000px;
top: 200px;
text-shadow: 2px 5px 5px #e3bce6;
}