Support » Plugin: footnotes » Update breaks layout

  • Resolved a223123131

    (@a223123131)


    After 2.5.9 was breaking my entire site I switched back to 2.5.8 and now the layout of the footnote is changed. After many many month with many many updates which broke things I’m realy pissed now!!!!!

Viewing 3 replies - 1 through 3 (of 3 total)
  • Thread Starter a223123131

    (@a223123131)

    I made an huge mistake… just did a quick check after the update from 2.5.7 to 2.5.8 and missed another bug. So going back to 2.5.7 now FOREVER !!!

    • This reply was modified 1 month, 2 weeks ago by a223123131.
    Plugin Author Mark Cheret

    (@markcheret)

    Hi @a223123131,

    thank you for raising your concerns.

    I’m sorry to hear that you’re pissed with the plugin. I wish you can give us another chance, we’re working really hard to make the plugin great: Performant, stable and useful.

    I wish you can pour some of your energy into advancing the development of the plugin by contributing bugfixes and enhancements: https://github.com/markcheret/footnotes

    And may I ask you to refrain from advising people to stop receiving updates? We’re gearing up to make footnotes more stable and useful and it would be a disadvantage for our users if they wouldn’t receive those updates.

    Best wishes
    Mark

    Plugin Contributor pewgeuges

    (@pewgeuges)

    @a223123131,

    I can only reiterate our apologies and beg your pardon for the Stable Tag mishap that led to the accidental release of a broken development version resulting from a total overhaul of all PHP files to get the code to comply to the rules that WordPress issued for all developers in the WordPress ecosystem to comply to.

    Plugin Author @markcheret, who posted our apologies in response to your support topic, published them also in a dedicated thread:

    https://wordpress.org/support/topic/2-5-10-reverts-2-5-9d1-and-apologies/

    And our new Plugin Author @rumperuu, who crafted the WordPress Coding Standards compliance update over in Footnotes’ new GitHub repository, published a detailed incident report:

    https://wordpress.org/support/topic/2-5-10-reverts-2-5-9d1-and-apologies/#post-14119440

    A few things that I posted in response have been deleted, among which the post where I addressed your other bug report:

    I made an huge mistake… just did a quick check after the update from 2.5.7 to 2.5.8 and missed another bug. So going back to 2.5.7 now FOREVER !!!

    I’m sad to read that you encounter a new bug in 2.5.8 that was not in 2.5.7. I swiftly diffed both versions one against the other two days ago, one day after the accident, and brought to your attention (in a meanwhile “archived” post) that there is now such new bug.

    Let’s go through the differences list together and see what changed:

    
    diff -wr footnotes_2.5.7/css/dev-common.css footnotes_2.5.8/css/dev-common.css
    9,10c9,10
    <  * @version 2.5.6
    <  * @lastmodified 2021-02-19T1523+0100
    ---
    >  * @version 2.5.8
    >  * @lastmodified 2021-02-28T1320+0100
    14c14,16
    <  *
    ---
    >  */
    > 
    > /**
    233,320d234
    < /**
    <  * Tooltips
    <  * 
    <  * - Bugfix: Tooltips: set z-index to maximum 2147483647 to address display issues with overlay content, thanks to @russianicons bug report.
    <  * 
    <  * @since 2.1.6
    <  * @reporter @russianicons
    <  * @link https://wordpress.org/support/topic/counter-styles-not-working/#post-13767299
    <  */
    < 
    < .footnote_tooltip {
    < 	display: none;
    < 	z-index: 2147483647 !important;
    < 	cursor: auto;
    < 	text-align: left;
    < 	padding: 12px;
    < 	line-height: 1.2;
    < 	font-weight: normal;
    < 	font-style: normal;
    < }
    < 
    < /**
    <  * Alternative tooltips
    <  * 
    <  * These default position values may be overridden by internal CSS.
    <  */
    < 
    < .footnote_referrer.relative {
    < 	position: relative;
    < }
    < 
    < .footnote_tooltip.position {
    < 	display: unset;
    < 	position: absolute;
    < 	bottom: 24px;
    < 	left: -50px;
    < 	width: 400px;
    < }
    < 
    < /*
    < fade-in parameters
    < */
    < 
    < .footnote_tooltip.shown {
    < 	visibility: visible;
    < 	opacity: 1;
    < 	transition-property: visibility opacity;
    < 	transition-timing-function: linear;
    < /*
    < property values of settings are inline CSS
    < 	transition-delay: 0ms;
    < 	transition-duration: 200ms;
    < */
    < }
    < 
    < /*
    < fade-out parameters
    < */
    < 
    < .footnote_tooltip.hidden {
    < 	visibility: hidden;
    < 	opacity: 0;
    < 	transition-property: visibility opacity;
    < 	transition-timing-function: linear;
    < /*
    < property values of settings are inline CSS
    < 	transition-delay: 400ms;
    < 	transition-duration: 200ms;
    < */
    < }
    < 
    < /*
    < Read-on button
    < */
    < 
    < .footnote_tooltip_continue {
    < 	font-style: italic;
    < 	color: green;
    < 	text-decoration: none !important;
    < 	cursor: pointer;
    < 	white-space: nowrap;
    < }
    < 
    < .footnote_tooltip_continue:hover {
    < 	color: blue;
    < 	text-decoration: underline !important;
    < }
    < 
    362,363c276,283
    < /*
    < label
    ---
    > /**
    >  * Reference container label.
    >  * 
    >  * - Bugfix: Layout: support right-to-left writing direction by replacing remaining CSS values 'left' with 'start', thanks to @arahmanshaalan bug report.
    >  * 
    >  * @since 2.5.8
    >  * @reporter @arahmanshaalan
    >  * @link https://wordpress.org/support/topic/right-to-left-text-problem/
    383c303
    < 	text-align: left !important;
    ---
    > 	text-align: start !important;
    429,435c349,369
    < /*
    < footnotes
    < 
    < @since 2.0.0  borderless table cells, thanks to @ragonesi bug report
    < @see <https://wordpress.org/support/topic/thin-box-around-notes-in-reference-container/>
    < @since 2.0.1  enforce borderless text cells through !important property, thanks to @ragonesi bug report
    < @see <https://wordpress.org/support/topic/box-around-c-references-container/>
    ---
    > /**
    >  * Footnotes list.
    >  * 
    >  * - Bugfix: Reference container: no borders around footnotes, thanks to @ragonesi bug report.
    >  * 
    >  * @since 2.0.0
    >  * @reporter @ragonesi
    >  * @link https://wordpress.org/support/topic/thin-box-around-notes-in-reference-container/
    >  * 
    >  * - Bugfix: Reference container: enforce borderless table cells, thanks to @ragonesi bug report.
    >  * 
    >  * @since 2.0.1
    >  * @reporter @ragonesi
    >  * @link https://wordpress.org/support/topic/box-around-c-references-container/
    >  * 
    >  * - Bugfix: Layout: support right-to-left writing direction by replacing remaining CSS values 'left' with 'start', thanks to @arahmanshaalan bug report.
    >  * - Bugfix: Layout: support right-to-left writing direction by enabling mirrored paddings on HTML dir="rtl" pages, thanks to @arahmanshaalan bug report.
    >  * 
    >  * @since 2.5.8
    >  * @reporter @arahmanshaalan
    >  * @link https://wordpress.org/support/topic/right-to-left-text-problem/
    443c377
    < 	text-align: left !important;
    ---
    > 	text-align: start !important;
    445a380,386
    > }
    > 
    > html[dir="rtl"] .footnote_plugin_index,
    > html[dir="rtl"] .footnote_plugin_index_combi,
    > html[dir="rtl"] .footnote_plugin_symbol,
    > html[dir="rtl"] .footnote_plugin_text {
    > 	padding: 5px 0 10px 6px !important;
    diff -wr footnotes_2.5.7/css/dev-tooltips.css footnotes_2.5.8/css/dev-tooltips.css
    9c9
    <  * Additional stylesheet needed when tooltips are enabled.
    ---
    >  * Additional stylesheet needed when tooltips (jQuery or alternative) are enabled.
    11,14c11,12
    <  * It doesn’t matter if the tooltips are jQuery based or alternative.
    <  * @since 2.5.5
    <  * @version 2.5.5
    <  * @lastmodified 2021-02-18T2030+0100
    ---
    >  * @version 2.5.8
    >  * @lastmodified 2021-02-28T1302+0100
    17c15
    <  * 
    ---
    >  * @since 2.5.5
    21d18
    < 
    23c20
    <  * Tooltips
    ---
    >  * Tooltips.
    29a27,32
    >  * 
    >  * - Bugfix: Layout: support right-to-left writing direction by replacing remaining CSS values 'left' with 'start', thanks to @arahmanshaalan bug report.
    >  * 
    >  * @since 2.5.8
    >  * @reporter @arahmanshaalan
    >  * @link https://wordpress.org/support/topic/right-to-left-text-problem/
    36c39
    < 	text-align: left;
    ---
    > 	text-align: start !important;
    diff -wr footnotes_2.5.7/footnotes.php footnotes_2.5.8/footnotes.php
    7c7
    < 	Version: 2.5.7
    ---
    > 	Version: 2.5.8
    21c21
    <  * @lastmodified 2021-02-27T0227+0100
    ---
    >  * @lastmodified 2021-02-28T1345+0100
    23c23
    < define( 'C_STR_FOOTNOTES_VERSION', '2.5.7' );
    ---
    > define( 'C_STR_FOOTNOTES_VERSION', '2.5.8' );
    diff -wr footnotes_2.5.7/readme.txt footnotes_2.5.8/readme.txt
    2c2
    < Contributors: mark.cheret, lolzim, dartiss, docteurfitness, felipelavinz, martinneumannat, matkus2, meglio, misfist, rumperuu, spaceling, vonpiernik, pewgeuges
    ---
    > Contributors: mark.cheret, lolzim, rumperuu, aricura, misfist, ericakfranz, dartiss, docteurfitness, felipelavinz, martinneumannat, matkus2, meglio, spaceling, vonpiernik, pewgeuges
    7c7
    < Stable Tag: 2.5.7
    ---
    > Stable Tag: 2.5.8
    18c18
    < **footnotes** aims to be the all-in-one solution for displaying an automatically generated list of references on your Page or Post. The Plugin ships with a set of current defaults but also empowers you to control how your footnotes are being displayed.
    ---
    > **footnotes** aims to be the all-in-one solution for displaying an automatically generated list of references on your Page or Post. The Plugin ships with a set of defaults while also empowering you to control how your footnotes are being displayed.
    79a80,83
    > = 2.5.8 =
    > - Bugfix: Layout: support right-to-left writing direction by replacing remaining CSS values 'left' with 'start', thanks to @arahmanshaalan bug report.
    > - Bugfix: Layout: support right-to-left writing direction by enabling mirrored paddings on HTML dir="rtl" pages, thanks to @arahmanshaalan bug report.
    > 
    332c336
    < - Update: GDPR: Add jQuery UI from WordPress instead of third party.
    ---
    > - Update: Libraries: Load jQuery UI from WordPress, thanks to @check2020de issue report.
    372c376
    < - Bugfix: No borders around footnotes in the container
    ---
    > - Bugfix: Reference container: no borders around footnotes, thanks to @ragonesi bug report.
    

    You see that many style rules for tooltips have been deleted. I need to apologize to @docteurfitness for overlooking that these remained in the common stylesheet when they were moved to dedicated stylesheets that are concatenated, minified and loaded only if tooltips are enabled.

    After that mistake, whose correction did not cause your bug but realized the performance improvement as designed, you notice a text-align: left; replaced with text-align: start;. For left-to-right scripts, this has no effect, whereas it debugs the layout of right-to-left scripts.

    Later below, the same again, plus mirrored paddings. The mirrored paddings are valid only if the root element has the right-to-left attribute. This debugs the layout of the reference container.

    Then, the tooltips are debugged in turn by replacing again left with start.

    Well, that’s all. The rest are comments, version numbers and changelog items, nothing code-related.

    As a consequence, the current version 2.5.10, equal to 2.5.8, renumbered to easier get rid of 2.5.9d1, is the best that Footnotes was ever able to deliver.

    We hope that this restores your confidence, and we beg your pardon, again.

    Best regards,
    @pewgeuges

Viewing 3 replies - 1 through 3 (of 3 total)
  • You must be logged in to reply to this topic.