This repository was archived by the owner on Oct 29, 2020. It is now read-only.
Static content misc#1549
Merged
weerd merged 2 commits intoDoSomethingArchive:devfrom Mar 27, 2014
weerd:static-content--misc
Merged
Static content misc#1549weerd merged 2 commits intoDoSomethingArchive:devfrom weerd:static-content--misc
weerd merged 2 commits intoDoSomethingArchive:devfrom
weerd:static-content--misc
Conversation
added 2 commits
March 27, 2014 11:20
…ut the styles and unify markup.
Contributor
There was a problem hiding this comment.
Semantically, this makes sense but it brings the front-end terminology out of sync with the field/variable names.
Should we update this in the module as well?
Contributor
Author
|
We could update the module as well (at least the variable), but updating the fields in the DB might get hairy... |
Contributor
|
agree, easy enough to just change the preprocess vars to match. |
Contributor
Author
|
Cool. I can add another issue for me to update the preprocess vars in the .module :) |
Contributor
|
🎯 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Need to double-check the confirmation and explore-campaigns pages since I'm not sure how to best view them on local. The explore-campaigns page seems to be generated by Views, but I don't see the template being used in Paraneue_DS.
Resolves #1515 (most of it anyhoo).