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

Manual (peer review + credits.php table of content + hb footer #432

Merged
merged 11 commits into from
May 23, 2020
Merged

Manual (peer review + credits.php table of content + hb footer #432

merged 11 commits into from
May 23, 2020

Conversation

michihdeu
Copy link
Contributor

No description provided.

Copy link
Contributor

@jteresco jteresco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's a long time ago and I don't remember any details, but I seem to recall there being some reason we didn't include the footer on pages like the HB and Mapview. It might have just been to maximize available browser real estate for the maps themselves, but there might have been some bad interaction when we had maps that try to fill the window. I'd like to hold off on this for the moment until I have a chance to test it on tmtest (which is currently the very active test area for the new Mapview functionality).

@michihdeu
Copy link
Contributor Author

I found this: #107

I've undone this change. Please bring the remaining changes live!

@jteresco jteresco merged commit 8670891 into TravelMapping:master May 23, 2020
@jteresco
Copy link
Contributor

Thanks - this is in and live on the main site.

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