-
Notifications
You must be signed in to change notification settings - Fork 44
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
Issue 26 #50
Issue 26 #50
Changes from all commits
9f02053
8397944
19f29f2
0271931
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
[ZoneTransfer] | ||
ZoneId=3 | ||
HostUrl=https://www.figma.com/ |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,98 @@ | ||
* { | ||
box-sizing: border-box; | ||
width: 100%; | ||
} | ||
|
||
html { | ||
font-size: 10px; | ||
} | ||
Comment on lines
+1
to
+8
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Vc não pode fazer isso num arquivo interno de There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. blz |
||
|
||
.organize { | ||
width: 100%; | ||
height: auto; | ||
display: flex; | ||
flex-direction: column; | ||
align-items: center; | ||
} | ||
|
||
#bonhoeffer { | ||
display: flex; | ||
flex-direction: column; | ||
} | ||
|
||
.question { | ||
font-family: Montserrat; | ||
font-size: 1.8rem; | ||
font-weight: 500; | ||
line-height: 26px; | ||
letter-spacing: 0px; | ||
text-align: center; | ||
color: #39434F; | ||
} | ||
|
||
#adjuste { | ||
font-family: Montserrat; | ||
font-size: 1.4rem; | ||
font-weight: 400; | ||
line-height: 0; | ||
letter-spacing: 0px; | ||
text-align: center; | ||
color: #606873; | ||
} | ||
|
||
#weight { | ||
font-family: Montserrat; | ||
font-size: 8rem; | ||
font-weight: 700; | ||
line-height: 50px; | ||
letter-spacing: 0px; | ||
text-align: center; | ||
color: #1B85F3; | ||
margin-bottom: 120px; | ||
} | ||
|
||
#rangeSlider { | ||
margin-top: 0; | ||
width: 600px; | ||
-webkit-appearance: none; | ||
} | ||
#rangeSlider:focus { | ||
outline: none; | ||
} | ||
#rangeSlider::-webkit-slider-runnable-track { | ||
width: 10000px; | ||
height: 1px; | ||
cursor: pointer; | ||
box-shadow: none; | ||
background: #606873; | ||
border-radius: 0px; | ||
border: 0px solid #010101; | ||
} | ||
#rangeSlider::-webkit-slider-thumb { | ||
box-shadow: none; | ||
border: 2px solid #1268CC; | ||
height: 42px; | ||
width: 75px; | ||
border-radius: 15px; | ||
background: #1B85F3; | ||
cursor: pointer; | ||
-webkit-appearance: none; | ||
margin-top: -20px; | ||
} | ||
|
||
#Slider-SVG { | ||
position: relative; | ||
display: flex; | ||
flex-direction: column; | ||
width: 100%; | ||
align-items: center; | ||
} | ||
|
||
#bars-svg { | ||
width: 100%; | ||
display: flex; | ||
background-repeat: repeat-x; | ||
position: absolute; | ||
top: -80px; | ||
z-index: -1; | ||
}/*# sourceMappingURL=index.css.map */ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Isso é um sourcemap? Pq tem um sourcemap no seu componente? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. po esse sourcemap eu não adicionei ai não, que brisa é essa |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Não tinha raparado antes, mas pq o arquivo se chama PetRegist? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Como ele vai fazer parte do cadastro de pets e ainda não existia nenhum arquivo, pensei em já deixar o nome para guiar o que esse componente faz parte. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,48 @@ | ||
/* eslint-disable prettier/prettier */ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. prettier e eslint estão dando conflito de identação, precisei desativar um para que não desse erro. Cheguei até a falar com o Pilu e Diogo sobre esse erro ai, e te mandei msg um dia no discord mostrando o erro, no fim ele era ocasionado por conta de conflito de identação |
||
import { Component } from 'pet-dex-utilities'; | ||
import './index.scss'; | ||
import Bars from './Bars.svg'; | ||
|
||
const html = ` | ||
<section class="slider"> | ||
<div class="slider__text-slider"> | ||
<h3 data-select="weight" class="slider__weight">0</h3> | ||
</div> | ||
<div class="slider__container"> | ||
<input type="range" value="50" step="0.1" min="0" max="100" data-select="rangeSlider" class="slider__input"/> | ||
<img class="slider__bars-svg" alt="rangeSlider" src="${Bars}" data-select="svg"> | ||
</div> | ||
</section> | ||
`; | ||
|
||
const events = ['weight']; | ||
|
||
export default function Slider() { | ||
Component.call(this, { html, events }); | ||
this.enable = true; | ||
|
||
this.selected.get('rangeSlider').addEventListener( | ||
'input', | ||
(event) => { | ||
if (!this.enable) return; | ||
|
||
this.weight(); | ||
this.setValue(event.target.value); | ||
}, | ||
false, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Pq esse false aqui? |
||
); | ||
} | ||
Slider.prototype = Object.assign(Slider.prototype, Component.prototype, { | ||
setTitle() { | ||
this.selected.get('weight').textContent = Text; | ||
}, | ||
setValue() { | ||
this.selected.get('weight').textContent = this.selected.get('rangeSlider').value; | ||
}, | ||
weight() { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. mudar para There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||
this.emit('weight'); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. o nome do evento muda pra |
||
}, | ||
disable() { | ||
this.enable = false; | ||
}, | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isso aqui não existe, vc fez em
css
? A gente usascss
nãocss
.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eu fiz em sccs.
foi uma extensão minha que criou esse .css, vou tirar ele de jogo, tinha esquecido.