-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/fos 55/events styling #381
Conversation
Please make the theme changes in ecc_theme |
I don't want to put any changes for the theme into ecc_theme until that theme has gotten to live. |
Marking as blocked until shared theme gets to |
I've added "Add to calendar" functionality, so marking this as |
Thanks @Polynya, I've made those changes now. |
Thanks @markconroy. Can we keep your 4 PRs open for now, until we know what is going to be deployed to Prod first, Fostering or Theme Sharing. |
@markconroy the shared theme has finally be deployed to Prod. The Essex theme is no longer used and will be deleted from this repo (LP-59). Please move the styling from this PR to https://github.com/essexcountycouncil/ecc_theme If the styles are unique to Gov then put them in ecc_theme_gov but if they could be used by Intranet in the future then they should go in ecc_theme, the shared base theme. |
Closing as it has been superceded (style changes in ecc_theme, config changes in another commit) |
Adds some layout tweaks to events listing page, so it now looks like this: