measured / brandguide

BMS engine for brandguide
admin.brandguide.io
0 stars 0 forks source link

pushState #22

Open jenbennings opened 10 years ago

jenbennings commented 10 years ago

It looks like we're using pushState to update URLs, which is resulting in this (annoying) browser history behaviour:

Could we instead use replaceState to avoid this happening within the editor?

I may be jumping the gun on this one, so it would be good to hear your thoughts on whether you think history in this context is beneficial to the user or not.

haustraliaer commented 10 years ago

I've been thinking about this while working on hive the last few days. My opinion is that it's almost always useful to the user, even for modals sometimes if the interaction inside that modal is complex enough.

jenbennings commented 10 years ago

Fair enough. In that case we should implement informative titles:

client name • page title

Do we even need brandguide at the front? I like how GitHub (among others) simply rely on the favicon:

haustraliaer commented 10 years ago

Yeah I think that would be the best approach, functional titles will make that history list seem less arbitrary.