rebased onto f2b2258a3f16bf5a32f6726b45e15a9ef82c01c3
The wording in the footer is perfect! Why not put that in all the current themes?
Or do you have another set of text in mind for the generic themes?
The default theme already says "powered by pagure"
https://pagure.io/pagure/blob/master/f/pagure/themes/default/templates/theme.html#_41
I left the openSUSE theme alone, because i figured that the folks closer to that theme could update as appropriate.
@ryanlerch The openSUSE theme footer is riffed off the Fedora one, for lack of a better thing to put there, as @hellcp will confirm.
I think the new text you're proposing is actually more useful because it separates it all nicely and gives a nice link for filing a bug report if an issue is discovered.
@ryanlerch The openSUSE theme footer is riffed off the Fedora one, for lack of a better thing to put there, as @hellcp will confirm. I think the new text you're proposing is actually more useful because it separates it all nicely and gives a nice link for filing a bug report if an issue is discovered.
The file an issue one was purposely left out of the generic one. People can add such a link to their own themes if they wish. The link to the Pagure source is enough for the generic theme.
I want to keep this commit to just the fedora themes for now. Feel free to change and propose a PR for the generic or openSUSE themes.
rebased onto eb5b175c136cfc2eaa1eeeeb77cd65da82a9da56
Looks awesome to me, let's merge :)
Pull-Request has been merged by pingou