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

Edit Page: Always enable Librarian Mode #1934

Closed
mekarpeles opened this issue Feb 27, 2019 · 10 comments
Closed

Edit Page: Always enable Librarian Mode #1934

mekarpeles opened this issue Feb 27, 2019 · 10 comments
Labels
Affects: UI Issues with the web site's user interface. [managed] Good First Issue Easy issue. Good for newcomers. [managed] Theme: Editing Issues related to the user editing/wiki editing experience. [managed]

Comments

@mekarpeles
Copy link
Member

mekarpeles commented Feb 27, 2019

Description

What problem are we solving? What does the experience look like today? What are the symptoms?

feature request: remove enter "librarian mode" on /edit page such that librarian mode is always enabled

see: https://twitter.com/reclus23/status/1100751365847072774

@mekarpeles mekarpeles added Good First Issue Easy issue. Good for newcomers. [managed] Theme: Editing Issues related to the user editing/wiki editing experience. [managed] ui labels Feb 27, 2019
@cdrini
Copy link
Collaborator

cdrini commented Feb 27, 2019

I definitely don't want it removed entirely; I think it's really useful especially for hiding possibly "dangerous" fields. I think a better solution would be to make it persist between page loads, so if I enter "librarian mode", it will remain activated until I exit librarian mode (even if I navigate to different pages between).

@cdrini
Copy link
Collaborator

cdrini commented Feb 27, 2019

(also, man, that list of feature requests is spot on!)

@hornc hornc changed the title Edit Page: Always enable Library Mode Edit Page: Always enable Librarian Mode Feb 27, 2019
@seabelis
Copy link
Collaborator

This would be great. Would it be possible to have this as an preference setting associated with the user's account that they could activate?

@shreyansh23
Copy link
Contributor

Can I take this issue @mekarpeles

@mekarpeles
Copy link
Member Author

@shreyansh23 Yes please!

@mekarpeles
Copy link
Member Author

@seabelis I think it would be easier for it to just always be enabled (less work, seems like a better solution)

@shreyansh23
Copy link
Contributor

@mekarpeles, submitted the PR. Please review it.

@tfmorris
Copy link
Contributor

tfmorris commented Mar 5, 2019

"Librarian mode" is poorly named since it's not really a mode and it's not, as far as I know, restricted to users in the librarian's group. It's not uncommon for UIs to hide "advanced" features/fields, but default. Why are we abolishing this based on a single tweet?

Did anyone ask the Twitter user what field(s) they need from the advanced fields? Perhaps there's just a single one that they always need that we could consider moving to the default fields? Perhaps it could be traded for another less commonly needed field.

Perhaps we could make whether or not the advanced fields are shown by default a configurable user preference (as @seabelis suggested) or have the current display persist from page to page (as suggested by @cdrini )

It seems like this could use a little more thought.

@LeadSongDog
Copy link

Modal behaviour is generally a bad thing. Much of what hides in "librarian mode" is in need of attention, but doesn't get it because it is obscured. The edition name should never have been there. The "translation of" should just be the work id. The "translated from" should be autopopulated from the language of the original edition of that work (or just done away with), not a troublesome, slow, picklist text repopulated for every subsequent translated edition. All this would have long ago been fixed if more people saw it.

@tfmorris
Copy link
Contributor

tfmorris commented Mar 7, 2019

Modal behaviour is generally a bad thing.

If we're going to speak in broad generalities I could say, "Progress disclosure of complexity is generally a good thing." Heck, I could even provide a citation to a world-renowned UX designer.

@brad2014 brad2014 removed the easy label Apr 29, 2019
@brad2014 brad2014 added the Affects: UI Issues with the web site's user interface. [managed] label May 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Affects: UI Issues with the web site's user interface. [managed] Good First Issue Easy issue. Good for newcomers. [managed] Theme: Editing Issues related to the user editing/wiki editing experience. [managed]
Projects
None yet
Development

No branches or pull requests

7 participants