Skip to content
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

Make the footer width adjustable per page. #25

Merged
merged 1 commit into from
Dec 3, 2023

Conversation

drgrice1
Copy link

@drgrice1 drgrice1 commented Dec 3, 2023

This sets a default footerWidthClass stash value to col-12 in the ContentGenerator go method. Any module or template can modify that as needed to match its layout. This is done in
templates/ContentGenerator/Instructor/Index.html.ep, templates/ContentGenerator/Instructor/Stats/index.html.ep, and templates/ContentGenerator/Problem.html.ep.

Note that this can also be done directly in the perl code for any package that derives from the ContentGenerator as well (i.e., in a .pm file). To do so call $c->stash->{footerWidthClass} = '...'.

I have left the remaining pages for you to determine, since I am uncertain what you will want on some of them.

This sets a default `footerWidthClass` stash value to `col-12` in the
ContentGenerator `go` method.  Any module or template can modify that as
needed to match its layout.  This is done in
`templates/ContentGenerator/Instructor/Index.html.ep`,
`templates/ContentGenerator/Instructor/Stats/index.html.ep`, and
`templates/ContentGenerator/Problem.html.ep`.

Note that this can also be done directly in the perl code for any
package that derives from the ContentGenerator as well (i.e., in a `.pm`
file).  To do so call `$c->stash->{footerWidthClass} = '...'`.

I have left the remaining pages for you to determine, since I am
uncertain what you will want on some of them.
@Alex-Jordan Alex-Jordan merged commit 27f3f6f into Alex-Jordan:footer Dec 3, 2023
1 check passed
@drgrice1 drgrice1 deleted the footer-adjustable-width branch December 3, 2023 23:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants