-
Notifications
You must be signed in to change notification settings - Fork 43
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
224 create new table component #2166
base: main
Are you sure you want to change the base?
Conversation
I agree that this does seem very specific to the one use-case, in which case we should probably make that clear from the naming. I think it makes some sense to have the code for this pulled out from the page itself (to avoid the page being massive), but maybe just using a partial would suffice? I can't comment so much on the accessibility aspects (maybe we can ask about that on Slack), but it does at least seem usable via keyboard. |
b0dc379
to
00aa922
Compare
0967537
to
5537dec
Compare
84e2b5e
to
e20fa49
Compare
e20fa49
to
627477d
Compare
Trello
This is a first iteration of the new table component for the Publisher landing page. It’s still a work in progress but at a useful stage for an initial review. It still needs some custom styling but seems to work pretty much as expected. There are currently no other components associated with this repo so this creates a new component guide section which can be viewed locally at this URL: http://publisher.dev.gov.uk/component-guide/table
I had a few concerns emerge in my mind as I was working on it, as detailed below. Apart from that the code is just good enough to make it work and will need some tidying plus I’d need to add a Jasmine test for the JS.
Concerns referred to are: