Skip to content

Connection Banner: Fix path calculation for svg images#8483

Merged
oskosk merged 1 commit intomasterfrom
fix/paths-in-connection-banner-images
Jan 9, 2018
Merged

Connection Banner: Fix path calculation for svg images#8483
oskosk merged 1 commit intomasterfrom
fix/paths-in-connection-banner-images

Conversation

@oskosk
Copy link
Copy Markdown
Contributor

@oskosk oskosk commented Jan 8, 2018

Fixes connection banner images when Jetpack is run from a directory named other than jetpack.

Changes proposed in this Pull Request:

  • Replaces usage of dirname( __FILE__) for JETPACK__PLUGIN_FILE when calculating the URL path to some images in the connection banner shown in WordPress dashboard.

Testing instructions:

  • Check this PR using the Jetpack Beta plugin.
  • Visit WordPress' dashboard.
  • Confirm that images are not broken in all tabs.

Before

image

After

image

Proposed changelog entry for your changes:

Fixed images shown in the dashboard when Jetpack is installed in a directory naed different than jetpack

@oskosk oskosk added General [Status] Needs Changelog [Status] Needs Review This PR is ready for review. Bug When a feature is broken and / or not performing as intended labels Jan 8, 2018
@oskosk oskosk added this to the 5.8 milestone Jan 8, 2018
@oskosk oskosk requested a review from a team as a code owner January 8, 2018 17:58
@eliorivero eliorivero added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Jan 9, 2018
@eliorivero
Copy link
Copy Markdown
Contributor

Works and solves the issue 🐑

@oskosk oskosk merged commit 4f4c461 into master Jan 9, 2018
@oskosk oskosk deleted the fix/paths-in-connection-banner-images branch January 9, 2018 18:00
jeherve added a commit that referenced this pull request Jan 29, 2018
zinigor pushed a commit that referenced this pull request Jan 30, 2018
* Changelog 5.8: create base for changelog.

* Update 5.8 release post link

* fix 5.8 release date

* Updates to plugin description

* Changelog: add #8499

* Changelog: add #8506

* Changelog: add #8509

* Changelog: add #8516

* Changelog: add #8517

* Changelog: add #8523

* Changelog: add #8547

* Changelog: add #8496

* Changelog: add #8584

* Changelog: add #8595

* Changelog: add #8445

* Changelog: add #8431

* Changelog: add #8284

* Changelog: add #8270

* Changelog: add #8124

* Changelog: add #8581

* Changelog: add #8463

* Changelog: add #8568 (#8646)

* Updates to testing list and changelog

* Changelog: add #8443

* Changelog: add #8459

* Changelog: add #8469

* Changelog: add #8464

* Changelog: add #8478 and #8479

* Changelog: add #8483

* Changelog: add #8488

* Changelog: add #8513

* Changelog: add #8555

* Changelog: add #8565

* Changelog: add #8601

* Changelog: add #8612

* Changelog: add first pass at Search items.

* Changelog: add more info to help test Search.

* Changelog: add #8144

* Changelog: add #8313

* Changelog: add #8419

* Changelog: add #8465

* Changelog: add #8515

* Changelog: add #8587

* Changelog: add #8591

* Changelog: add #8659

* Changelog: add #8661

* Changelog: add #8671

* Changelog: add 5.7.1 to archived changelog too.

* Reverted changes to readme, removed entry about backups.
@kraftbj kraftbj removed the [Status] Ready to Merge Go ahead, you can push that green button! label Oct 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug When a feature is broken and / or not performing as intended General

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants