feat: started dasboard page dvt60/dashboards-page #96
Conversation
| { id: 1, title: 'ID', field: 'id', flex: 1, checkbox: true }, | ||
| { id: 2, title: 'Title', field: 'title', flex: 2 }, | ||
| { id: 3, title: 'Date', field: 'date', flex: 2 }, | ||
| ]; |
There was a problem hiding this comment.
dummyHeader dummy değil aslında gerçek başlıklar kodluyoruz "dummy" kaldırabilirsin sadece header ya da headerData olur, bu başlık 3 tane var ama figmadaki aynısı 7 tane başlık kalmalı
| bold | ||
| typeColour="powder" | ||
| onClick={() => {}} | ||
| /> |
There was a problem hiding this comment.
StyledDashboardListButtons height 56px kalmalı figmadaki height aynısı olmalı boşlukları eşit düzeltmeli
| <StyledDashboardListButtons> | ||
| <StyledDvtSelectButtons> | ||
| <DvtButton | ||
| label="Unselect All" |
There was a problem hiding this comment.
Unselect? figmadaki Deselect yazıyor. bu ne işe yarayacağı function gerekiyor
There was a problem hiding this comment.
tablodaki seçilmiş olursa sayısı "10 Selected" falan olması lazım. figmadaki yanısı olmalı
|
navbar altındaki gözükmemeli, dvt-navbar düzeltebilirsin figmadaki aynı olmalı |
| const handleCreateDashboard = () => { | ||
| history.push('/superset/dashboard'); | ||
| }; | ||
| console.log(data); |
| header={headerData} | ||
| selected={selectedRows} | ||
| setSelected={setSelectedRows} | ||
| checkboxActiveField="id" |
There was a problem hiding this comment.
senin ki data checkboxActiveField="id" olan data içinde id bulamadıysa sorun olur. Data içinde tek bir unique varsa o olmalı
| <DvtPagination | ||
| page={currentPage} | ||
| setPage={setCurrentPage} | ||
| itemSize={data.length} |
There was a problem hiding this comment.
itemSize={data.length} > data değil aslında api geliyorsa count koyman gerekiyor ve page 2 olursa api dönüştürmesi gerekiyor
| @@ -110,6 +110,8 @@ function DvtDashboardList() { | |||
| } | |||
| }; | |||
| fetchApi(); | |||
| setSelectedRows([]); | |||
| setSelectedItemCount(0); | |||
There was a problem hiding this comment.
setSelectedItemCount(0); bu gerek yok, page sıfırlanmasın
| function DvtDashboardList() { | ||
| const [selectedRows, setSelectedRows] = useState<any[]>([]); | ||
| const [currentPage, setCurrentPage] = useState<number>(1); | ||
| const [selectedItemCount, setSelectedItemCount] = useState<number>(0); |
There was a problem hiding this comment.
const [selectedItemCount, setSelectedItemCount] = useState(0); gerek yok kaldırabilirisn. SelectedRows.lenght kullanabilirisn yeterli
| <StyledDvtSelectButtons> | ||
| <StyledSelectedItem> | ||
| <StyledSelectedItemCount> | ||
| <span>{`${selectedItemCount} Selected`}</span> |
There was a problem hiding this comment.
selectedItemCount kaldırıyorsun, yerinde selectedRows.length kullanmalı.
SUMMARY
Dashboard page has been created.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION