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
Feat: add button on domains page #207
Conversation
Add button on the domains page that gets the user to new domain screen Closes #198
@@ -31,6 +32,11 @@ export default function DomainsIndexRoute() { | |||
been the industry's standard dummy text ever since the 1500s, when an unknown printer took a | |||
galley of type and scrambled it to make a type specimen book. | |||
</Text> | |||
<Flex justifyContent="flex-end"> | |||
<Link to="/domains/new"> | |||
<Button rightIcon={<AddIcon boxSize={3} />}>Create new domain</Button> |
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.
I wonder if Domain
(capital) is better? Not sure.
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.
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.
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.
I don't have a hard opinion on that, if you guts want I can change it to capital
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.
Same, we should just set a standard and stick with it.
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.
Looks great to me
Add button on the domains page that gets the user to new domain screen
Closes #198