Company logo not displayed on mobile

[davoraltman]

The company logo is not being displayed when [jobs] page is viewed on mobile.

Thanks for taking a look!

604526-zen

[spencerfinnell]

Which theme?

[davoraltman]

I was able to reproduce on Twenty Sixteen.

[davoraltman]

@jom Can you take a look at this one when you are back? Thanks!

[donnapep]

Looks like it was a conscious decision to hide the logo at smaller resolutions, probably to avoid breaking the layout - https://github.com/Automattic/WP-Job-Manager/blob/master/assets/css/frontend.less#L888. Do we want to show the logo at all resolutions now?

[davoraltman]

@donnapep I'd say we do. Let's see what others think.

cc @dbtlr @annezazu @bikedorkjon

[annezazu]

I have no real authoritative opinion :/ I wonder if a mobile dev or a dev who really understands mobile dev could offer an opinion. Mine would be offered on shaky ground.

[dbtlr]

I would need to evaluate some test cases to understand the pros and cons of showing vs hiding. @donnapep Do you think you could do some screenshots of this at various resolutions, with it shown and hidden, so we can see what it would look like?

[donnapep]

@dbtlr Sure thing.

Here's iPhone 6 with no images (i.e. what it currently looks like):

iPhone 6 with images:

iPad with images (i.e. what it currently looks like):

So the smaller resolutions would need some work. It's not just a matter of turning the visibility of the image back on.

[dbtlr]

@donnapep Yeah, it seems like if we rearranged it to put the title on the top and the meta on the bottom, there would be plenty of room for the default style of have images in it on the left.

[jom]

Closing but should inform our work on the new jobs block (#1287)