ManifoldScholar / manifold

Transforming scholarly publications into living digital works.
http://manifoldapp.org
GNU General Public License v3.0
237 stars 31 forks source link

Revisions to standalone header feature #2304

Closed dananjohnson closed 5 years ago

dananjohnson commented 5 years ago
zdavis commented 5 years ago

Comps: image

Implementation: image

tsmyre commented 5 years ago

The Standalone mode tool tip reads incorrectly at present:

If enabled, this project will always render in standalone mode when “&mode=standalone” is appended to the URL.

The ampersand should be a question mark: ?mode=standalone.

tsmyre commented 5 years ago

Also, when toggled to Enabled a project does not persist the ?mode=standalone in the URL when a user clicks on a text. For example, see this project: https://edge.manifoldapp.org/projects/the-adventures-of-sherlock-holmes?mode=standalone.

If you click on any of the texts and enter the reader, the menu you see will still provide all the options available for library view: Project Home, Projects, Following. If you nav back that way, Standalone is no longer visible.

Likewise, if you return fresh to https://edge.manifoldapp.org/projects/the-adventures-of-sherlock-holmes?mode=standalone and click on any resource, when you arrive at the resource page the standalone suffix will be gone, though the standalone header will persist. If you use that to nav back to the project home, the standalone header will be there even though the URL suffix is not showing. However if you click refresh, you'll be out of standalone.

zdavis commented 5 years ago

reader footer logo margin regression image

tsmyre commented 5 years ago

I'm wondering if there should be a default header bar color when a user doesn't specify one in settings. Right now the bar turns white and is only visible on hover if no color is provided.

tsmyre commented 5 years ago

I'm also wondering if it would be possible to have a toggle that allows a public project to be in standalone and not appear in the library. I suspect that will be a highly sought after means of conveying projects when in standalone.

dananjohnson commented 5 years ago

I'm wondering if there should be a default header bar color when a user doesn't specify one in settings. Right now the bar turns white and is only visible on hover if no color is provided.

Yeah there definitely needs to be a default color here. Without one specified, the BG is actually just transparent.

Screen Shot 2019-08-22 at 9 08 52 AM
zdavis commented 5 years ago

Could you implement this, Dana? I say we go with the manifold green as default.

dananjohnson commented 5 years ago

Can do! Although that green is also the hover color. I'm wondering if we should make green the default, but update the hover/focus state to just lighten or darken whatever the BG color is.

zdavis commented 5 years ago

I like that idea

dananjohnson commented 5 years ago

Comps:

Screen Shot 2019-08-26 at 10 31 50 AM

Current:

Screen Shot 2019-08-26 at 10 32 27 AM
zdavis commented 5 years ago

image

dananjohnson commented 5 years ago
Screen Shot 2019-08-28 at 9 42 44 AM

This is the checkStandaloneMode method in client/src/frontend/containers/ProjectWrapper/index.js.

dananjohnson commented 5 years ago

@zdavis I'm having trouble reproducing the issue with notification placement. The position looks right when I test it. Are you seeing it in all cases or just in certain circumstances?

zdavis commented 5 years ago

image