-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Refactor the commanders list #73
Comments
Hi @petrvecera I would love to work on this also for the hactoberfest, pls do assign it for me |
There is several helper function to get the information needed. These functions are in files in folder The assets are in folder Some additional things which can help : // Returns the full commander icon, more info in file packages/web/src/pages/commanders/commanderDetails.tsx
// This is the icon we want to show, probably we could scale it little bit down so it's not so big
getCommanderIconPath(commanderData.iconlarge) I would definitely look into file Each commander source data are saved in file:
Again in the commander detail, here is example how the abilities are shown. They are as a array on each commander so we could add them directly to list component: <List
itemLayout="horizontal"
dataSource={commanderData.abilities}
renderItem={(item: Record<string, any>) => (
<List.Item>
<List.Item.Meta
avatar={
<div>
<Avatar
alt={item.name}
src={getExportedIconPath(item.icon)}
shape="square"
size={64}
/>
<Badge count={0} overflowCount={999} showZero offset={[0, -32]}>
<></>
</Badge>
</div>
}
title={item.name}
description={item.description}
/>
</List.Item>
)}
/> |
Hi @poly4concept I wanted to check if you would have time to look into this or I should give it up for grabs for someone else. Thank you |
Fixed by #115 |
https://coh2stats.com/commanders/wermacht
Right now we have a list like this:
The goal would be to have a list similar to the list as shown here https://coh2index.com/army/west_german.html (bottom of page)
but still maintain the description ...
Example:
Tasks:
The text was updated successfully, but these errors were encountered: