Implement Pipelines page layout #203
Conversation
jandsonrj
added
enhancement
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #203 +/- ##
=======================================
Coverage 72.50% 72.50%
=======================================
Files 24 24
Lines 811 811
=======================================
Hits 588 588
Misses 223 223 ☔ View full report in Codecov by Sentry. |
… from the general background color.
frontend/styles/pages/banner.js
Outdated
| @@ -0,0 +1,51 @@ | |||
| import { makeStyles } from '@mui/styles' | |||
|
|
|||
| const useStyles = makeStyles(theme => ({ | |||
There was a problem hiding this comment.
@jandsonrj é melhor substitui o makeStyle pelo sx ou styled componente a versão nova do material está abandonando o makestyle.
Sugiro o uso sx ao inves de ter um arquivo com todos os css coloca o que seria classe direto no atributo sx do componente.
There was a problem hiding this comment.
This file is out of use, I just forgot to remove it. Now, it has been removed!
frontend/pages/specz_catalogs.js
Outdated
|
|
||
| function SpeczCatalogs() { | ||
| return ( | ||
| <div style={{ textAlign: 'center' }}> |
There was a problem hiding this comment.
Como estamos usando material ui, ao inves de criar uma div com style pode usar o Box com sx no final é a mesma coisa, mas vc vai estar usando a sintaxe do material ui
caso precisemos alterar alguma coisa no futuro todos os componentes vão estar uniformes
There was a problem hiding this comment.
Okay, I made the suggested changes.
I put 'div' because it is a temporary screen, as I am already developing the main screen. Thank you for the tips.
I need you to close this PR urgently so that I can continue with the new screens from this update