Skip to content
This repository has been archived by the owner on Oct 29, 2019. It is now read-only.

Override width and height property and logic. Show help text for override fields #52

Merged
merged 2 commits into from
Nov 13, 2015

Conversation

cyberstar
Copy link
Contributor

  • Fields to keep settings for image,
  • Django 1.7 migrations
  • Django 1.7 migrations
  • Fix for help text (css) to be visible for override fields
  • Some logic to use override fields values instead of device specific values

Note that if aspect ration is selected - height would be calculated based on aspect ratio.

Note2 Django 1.7 migration (0003) also picked other changes to the fields, so that would be in sync with the code.

@FinalAngel
Copy link
Member

FE LGTM :)

null=True,
help_text=_(
'if this field is provided - it would be used across all devices '
'instead of default for devices types.'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the srcset with different sizes for different devices is not implemented yet. (I've never finished it, but it is 80% done)

@stefanfoulis
Copy link
Contributor

except for the help texts, LGTM

cyberstar pushed a commit that referenced this pull request Nov 13, 2015
Override width and height property and logic. Show help text for override fields
@cyberstar cyberstar merged commit 97aa094 into develop Nov 13, 2015
@cyberstar cyberstar deleted the feature/override-image-width-height branch November 13, 2015 15:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants