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

Largo prevents saving new default image widths #404

Closed
benlk opened this issue Feb 13, 2015 · 3 comments
Closed

Largo prevents saving new default image widths #404

benlk opened this issue Feb 13, 2015 · 3 comments
Assignees
Labels
priority: normal Must be completed before release of this version of plugin. status: in progress type: improvement

Comments

@benlk
Copy link
Collaborator

benlk commented Feb 13, 2015

Setting a custom image width in Dashboard > Settings > Media will not be respected by Largo. When using Largo, the values are set to their Largo defaults:

  • Thumbnail: cropped to 140 x 140
  • Medium: 336 x 9999
  • Large: 771 x 9999

Changing to the Twenty Twelve theme shows the values that had been manually set.

Changing the values in functions.php does not appear to affect the displayed values.

@aschweigert
Copy link

Yeah, that's intentional, of course but we should probably either hide those fields or somehow add a note that these are not the settings people are looking for.

@benlk
Copy link
Collaborator Author

benlk commented Feb 13, 2015

Issue #404: bug not found.

We could hide that specific part of the settings with:

function hide_media_options() {
  remove_menu_page( 'options-media.php' );        //Settings 
}
add_action( 'admin_menu', 'hide_media_options' );

@aschweigert
Copy link

right. I think that would be ok. anyone have any reservations re: going down that road? @rnagle?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: normal Must be completed before release of this version of plugin. status: in progress type: improvement
Projects
None yet
Development

No branches or pull requests

3 participants