Twenty Eleven: "Add Citation" Text Color issue in Pullquote Block

[kajalgohel]

In Twenty Eleven Theme, when we add Pullquote block in editor side, and choose text color, We can see that the text color is reflected for "Add quote" and "Add citation" text.But when we see the same Pullquote block at front side, text color for "Add citation" text is not reflected.

Steps to replicate:
1: Activate the Twenty Eleven Theme
2: Add Pullquote block
3: Enter some Text for "Add quote"
4: Enter some Text for "Add citation"
6: Choose Text color
7: Save Page/Post
8: View the page/post at front side

For better understanding I provide video attachment link.
Video URL : ​https://share.cleanshot.com/wZpuZo9ZCN9xpSHzIb7s

Thanks

[kajalgohel]

patch added

[karmatosed]

Without patch

[karmatosed]

With patch

[karmatosed]

I can replicate the issue and the patch does fix it. This means for me that I would flag for commit as it is a smaller patch which does resolve the issue.

Thank you for your collaboration on this @kajalgohel.

[karmatosed]

In 57896:

Twenty Eleven: Fixes adding citation text color issue in pullquote block.

The pullquote block wasn't reflecting text color added for quote and add citation text across the front and back of editor. This fixes that by adding ensuring whatever color is selected is shown in both

Props kajalgohel.
Fixes #56524.

[sabernhardt]

before: citation was set to #141412 when the block had the default text color or the theme's blue

[sabernhardt]

after: citation inherits #373737 to match the paragraph when no text color is selected, and it inherits the theme's blue when chosen for the Pullquote block

[sabernhardt]

Inheriting the text color affects the citation color when the user does not select a color for the block, yet in this case it seems reasonable.

• [43794] only added the #141412 color in blocks.css, not editor-blocks.css, and I did not find that color anywhere else in Twenty Eleven.
• The difference between #141412 and #373737 is minor with a ratio of 1.54:1 (or against a white background, the ratios are 18.44:1 for #141412 and 11.9:1 for #373737).
• The code change in [57896] is simpler than adjusting the color only when the block has the .has-text-color and/or .has-background classes.

For the screenshots, I edited the code element font-size and/or text-transform to make the inline code more readable. The colors and other styles should be accurate.

[karmatosed]

@sabernhardt I am unclear are you requesting the commit gets reverted or suggesting some iterations? I am totally happy with either of those options but trying to understand the next steps as it has been committed.

[sabernhardt]

I do not think it is worth reverting the change. Any similar tickets may require a different strategy, but the new default color seems acceptable for this theme.

[karmatosed]

Thank you for confirmation @sabernhardt. I agree it's tricky with these tickets. Adding stacks like this to block.css front/back isn't the best workaround situation. Thank you for the diligence in checking.