The project page should mention that you have to disable, then enable, your theme in order to get breakpoints read in from the .info file.

CommentFileSizeAuthor
#8 breakpoints-documentation-clarification-2075595-8.patch4.63 KBjwaxo

Issue fork breakpoints-2075595

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

WarrenK’s picture

WarrenK CreditAttribution: WarrenK commented

Documentation should also reference https://drupal.org/node/1869968 where attiks wrote, "At the bottom of the theme settings page you should see a new button to reload the theme info, do you see it?"

attiks’s picture

attiks CreditAttribution: attiks commented
Assigned: Unassigned » Jelle_S
Issue summary: View changes
attiks’s picture

attiks CreditAttribution: attiks commented

From[#1869968]

A mention of where to find this button in the help text on admin/config/media/breakpoints would suffice to fix this UX bug.

hass’s picture

hass CreditAttribution: hass commented

I think we do not need to document broken logic #1869968: Can't add/delete theme breakpoints that need to be fixed. That is not worth a doc.

attiks’s picture

attiks CreditAttribution: attiks commented

The documentation is adding a sentence like 'at the bottom of the theme settings page you can find a button to reload the breakpoints of yout theme ...."

Jelle_S’s picture

Jelle_S
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: Jelle_S commented
Version: 7.x-1.1 » 7.x-1.x-dev
Assigned: Jelle_S » Unassigned
Status: Active » Fixed

Fixed in latest dev version.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

jwaxo’s picture

jwaxo CreditAttribution: jwaxo commented
FileSize
breakpoints-documentation-clarification-2075595-8.patch4.63 KB

Hello,

As I wasn't able to find this information in the included documentation with the latest dev version, I completely revamped the very minimal README.txt with a newer one, which also includes this information about reloading the theme when updating breakpoints via the .info file.

Attached is a patch to update README.

I tried to cover a lot of other information that doesn't appear to be laid out anywhere. I'm also trying to update a lot of popular contrib modules as a whole, so if you have any commentary on the documentation style I have laid out here, let me know.

Jelle_S’s picture

Jelle_S
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: Jelle_S commented
Status: Closed (fixed) » Active
Jelle_S’s picture

Jelle_S
Primary language Dutch
Location Antwerp, Belgium
CreditAttribution: Jelle_S commented
Status: Active » Fixed

Fixed in latest dev version. Thanks for the patch!

  • Jelle_S committed 49fcad0 on 7.x-1.x authored by jwaxo 
    Issue #2075595 by jwaxo | WarrenK: Fixed Documentation needs...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

dfgdfgd made their first commit to this issue’s fork.