-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feedback on mobile-app UI as of commit d4148926cc. #33
Comments
I think it is broken for me. It looks like a site that loaded but the css didn't come through. I don't see any 400-499 html responses except 404 on a font (https://arches-app-demo.opentechstrategies.com/assets/Inter-VariableFont_opsz_wght-c8O0ljhh.ttf) and a favicon. |
@jvasile this is the mobile layout only, following @reefdog's mobile layout design It should translate a good amount better if you reduce your window size to a mobile panel, working on the desktop layout now, will ping when that is merged! |
Looking at it on my phone, it looks much better! I really like being able to scroll through the pictures. It is simple and usable. My feedback is a pile of small things:
And finally, a bigger piece of feedback, which is less about design and more about vision. The site is exactly what it should be today. But tomorrow we can do better. Today, the collection is neat, and you can walk through some links in the same way wikipedia affords. That's cool, but it doesn't tell any stories. There's no curation. Its joy is all serendipitous (i.e. haphazard). That's the thing I'd like to see change over time, because we won't engage users if we don't give them curated paths in. The site right now is a reference work, but it eventually needs to be a coffee table book or a comic book or a zine. All in all: well done on the design. This is tremendous progress. |
+1 to James's point about future direction, btw -- I agree, we eventually to get somewhere that is more like a coffee table book or a comic book or a zine. But just getting to a clean, uncurated, neutral reference site today is great. |
This is feedback on the currently-mobile-UI-only Wabash Arts Corridor demo Arches App. However, most of this feedback is not specific to screen size and will apply just as much to the upcoming desktop version of the app.
This feedback corresponds to this internal Zulip message.
I tested on a Google Pixel 7a, in both Firefox (nightly) and in the default Chrome browser that ships with Google's stock Android.
Title at top: "Explore Chicago's Wabash Arts Corridor". #58
Then put the current top intro text paragraph behind an "🛈" link or something. Justification: we shouldn't start out the site with big blob of text that drives all the action controls down such that they're not even initially visible on the landing screen.
Also, if we are quoting something, then we should cite it. I think the text we have comes from an old version of https://students.colum.edu/deps/wabash-arts-corridor, that was snapshotted by Google and attributed to the alias link https://wabashartscorridor.org/page/2/ (which is a 404 these days, but top-level https://wabashartscorridor.org/ redirects to https://students.colum.edu/deps/wabash-arts-corridor).
Feel free to use/remix the more up-to-date text from the new site (or I can do so if you'd prefer).
Remove the text "Explore the artwork geographically". The map+pins signify that well enough that no explanation is needed.
Remove the text "Explore the artwork as a list". #59
Make initial map taller, so pins aren't so squished together. The Wabash Arts Corridor runs mostly North<->South :-). #60
For "No image available", use gray background (see PR #32).
Instead of "Everything Artists Artworks Photographers Structures" above the search box, just have "Artworks Artists" as the only two choices, with "Artworks" selected as the default.
Then for the placeholder text inside the search box, have "Search artwork titles and artist names..." #61
When crediting the photographer (e.g., under the image, on a specific artwork's result page), don't link to the photographer as an entity. Just have "Photographer: Rose McGowan" with a colon but no link (well, obviously use whatever the entity's name is, but as of right now all the photos are taken by Rose McGowan).
Also on a specific result page, typographically distinguish the label "WAC Description" from the actual description text by making the latter italic (as well as in double quotes, which it already is). If you think some other way of distinguishing works better than italicizing, then try that -- I'm just suggesting italicization as a default solution, but I'm not wedded to it.
For the "Arches metadata": put a colon after that, then underneath, indent the metadata (not textual indent, but rather I mean move the whole box's left edge over a bit to the right, so that even the continuation lines of long UUIDs are still indented).
When I click on an address, I expected that to do something on the map, like zoom me in to that address, as well as showing what Artwork is there. But nothing happens on the map. Can we connect address links to appropriate map actions? (This feedback item may be non-trivial and require some discussion -- let me know.)
Clarify resource type in search list #73
Also in the text for each result, include the address where the artwork is located (that is, the artwork whose image is being shown, or whose image would be shown but for "no image available"). Likewise, distinguish this address text typographically and spatially from the
Introduce Client Side Routing #57
Then I hit my browser's Back button to go back to the front page, but that didn't work -- instead it took me to the page I'd been on before I went to the WAC demo app in the first place. (So then I used the browser's Forward button to go back to the search result, and I saw the "Back to search" link and used that... but people are going to use their Back button reflexively, so that has to Just Work.)
For the "About / Credit" text at the bottom (I haven't tested this HTML, so please fix as needed):
That "SEE_BELOW" link should link to a "/technical-info" page that has the following content:
Did I get all the above URLs right? Please correct if any are wrong, and let me know of course.
The text was updated successfully, but these errors were encountered: