The select box (text format) for Add Content and Add Comment differ in styling. I have tested on Firefox (Linux). Also screenshots have been attached.
Create bug:
1. Use Bartik theme as administration theme.
2. Visit Create Article from Content Section.
3. Below the body section, to select Text Format you will see this
4. Submit the article.
5. Create a comment for the article.
6. In the text format select box for comment body you will see this
Both these Select boxex should have same styling.
Comment | File | Size | Author |
---|---|---|---|
#41 | 2826903-after_patch-2-#39.png | 73.85 KB | BhumikaVarshney |
#41 | 2826903-after_patch-1-#39.png | 80.71 KB | BhumikaVarshney |
#40 | 2826903-after_patch-2.png | 20.02 KB | Abhijith S |
#40 | 2826903-after_patch-1.png | 25.76 KB | Abhijith S |
#39 | 2826903-38.patch | 432 bytes | djsagar |
Issue fork bartik-2826903
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:
Comments
Comment #2
kostyashupenkoI've just removed border-radius at this patch to make it absolutely 100% as on node create/edit pages. But i think we need more description about it, coz looks like on your first screen it is seven theme for administration theme, so i can't understand the phrase "1. Use Bartik theme to setup site."
On my screen there is bartik theme as default and as administration theme and it looks ok for now on my side after this little patch. But if you talk about bartik - default theme, and seven - administration theme => in this case i think we don't need any changes, coz selectboxes at administration theme doesn't have any styles. So styles are default, just related to browsers (everywhere it's different).
I'll resume. So:
1. I think we need more clear description of this task
2. Maybe more screens
Now how it looks at my FF (linux as well) after this patch
Comment #3
codehimanshu CreditAttribution: codehimanshu commented@kostyashupenko
This is not what it looks like on my screen even after applying patch.
Even before applying patch, the font-family of both the pages node create comment and add article is different. I have attatched the screenshot of both the pages for FF(linux). This is after applying your patch.
Comment #4
neerajpandey CreditAttribution: neerajpandey commentedIn my case the dropdown button in comment body has a border-radius but there there is no such thing in the article body.
So I think this should work.
Comment #5
denutkarsh CreditAttribution: denutkarsh at Google Code-In commented@neerajpandey Thanks for picking this up. I tested your patch and it looks like the patch cannot be applied to drupal 8.2.x . Also I would advise you to apply the patch locally before posting it to the d.o , it saves a lot of time.. Or else you can use dreditor.
Comment #7
denutkarsh CreditAttribution: denutkarsh at Google Code-In commented@neerajpandey Try this patch, i think it does what you wanted to do. I have added you as author while submitting the patch.
Comment #8
denutkarsh CreditAttribution: denutkarsh at Google Code-In commentedComment #9
denutkarsh CreditAttribution: denutkarsh at Google Code-In commentedHere are the screenshots after applying the patch in comment 7. According to this issue both these select boxes should have the same styling and this patch actually solves the issue.
Comment #10
Thew CreditAttribution: Thew at Google Code-In commentedI tested, confirmed this bug.
Also, tested that the patch changes the radius of the comment form.
Change issue summary
Change the step one of how to create bug.
Comment #12
denutkarsh CreditAttribution: denutkarsh at Google Code-In commentedUnrelated random fail b/c #2828559: UpdatePathTestBase tests randomly failing. The patch failed because of this, there is no problem with the patch. I am setting the issue as fixed.
Comment #13
denutkarsh CreditAttribution: denutkarsh at Google Code-In commentedComment #15
denutkarsh CreditAttribution: denutkarsh at Google Code-In commentedAgain settings this to RTBC as the fails were unrelated.
Comment #16
alexpottSo I think this needs more thought. Firstly I'm not sure I agree with the premise that the boxes should look the same. But anyway... even if so..
in bartik's form css is purposefully styling it like this.
Also this "issue" if it one can be caused by unticking the checkbox for "Use the administration theme when editing or creating content" on admin/appearance
Comment #18
surbhiG CreditAttribution: surbhiG commentedIn this patch i have removed the border radius from the select field instead of overriding the border radius for select field and tested it also by unticking the checkbox for "Use the administration theme when editing or creating content" on admin/appearance.
Comment #19
Vidushi Mehta CreditAttribution: Vidushi Mehta at gai Technologies Pvt Ltd commentedI've checked on my machine in FF and both selects looks same didn't found any difference. Adding screenshots for the same.
Comment #20
legovaerComment #21
sheavey CreditAttribution: sheavey as a volunteer commentedI tested the most recent patch on on a fresh instance and the issue appears to be still be active. The Text format select box is still displaying erroneously on the comment form doesn't match its counterpart on the Article creation form.
I am running the following specs:
Drupal Vers.: 8.4.0-dev
OS Vers.: macOS Sierra 10.12.4
Chrome Vers.: 58.0.3029
Firefox Vers.: 53.0 (64-bit)
Safari Vers.: Version 10.1 (12603.1.30.0.34)
I'm attaching screenshots from both Chrome and Firefox.
At this point I think the issue is still open and I will work on a new solution over the next few days.
Thank you for your time.
Comment #22
cilefen CreditAttribution: cilefen commentedHi sheavey:
I found this issue on the triage spreadsheet for Baltimore while I've been assigning issue credit. Thank you for helping out. It looks as though you got partway through a triage. I could give you credit if you complete and document all triage steps. Thank you!
Comment #23
poojakural CreditAttribution: poojakural at Srijan | A Material+ Company commentedIssue tested on window and mac machine. it is working fine. Here are the attached screenshots.
@sheavey I have tested issue on mac firefox browser but could not replicate. I am using 53.0.2. What is yours?
Comment #24
poojakural CreditAttribution: poojakural at Srijan | A Material+ Company commentedI am moving to RTBC as could not replicate this issue.
@sheavey if you still facing issue and mark this issue as need work status
Comment #25
Gábor HojtsyReading through the issue summary I don't understand how does the patch address the problem? Would moving the border radius to not apply to the select box restore the original styling of the element?
Comment #26
Wim LeersPer #25.
Comment #30
pawandubey CreditAttribution: pawandubey as a volunteer and at TATA Consultancy Services commentedIn the comment form, I have applied the style in
<select>
element as similar to admin seven theme. After this patch you can find the same style as per the attached screenshot.Please review this patch and let me know your feedback.
Comment #31
pawandubey CreditAttribution: pawandubey as a volunteer and at TATA Consultancy Services commentedComment #34
tanubansal CreditAttribution: tanubansal at Salsa Digital commentedComment #35
tanubansal CreditAttribution: tanubansal at Salsa Digital commented@pawandubey : I have added the latest patch that you have provided. Tested on Firefox, chrome and other browsers.
Problem still exists when we change theme to 'Bartik' and create article content.
Text format select box for comment body still have different styling on Firefox.
Comment #36
poojakural CreditAttribution: poojakural at Srijan | A Material+ Company commentedComment #38
djsagar CreditAttribution: djsagar at OpenSense Labs commentedComment #39
djsagar CreditAttribution: djsagar at OpenSense Labs commentedUpdating patch for drupal 9.2.x-dev.
Please review.
Thanks!
Comment #40
Abhijith S CreditAttribution: Abhijith S as a volunteer and at Zyxware Technologies commentedApplied patch #39 on 9.2.x and it works fine.The selectbox styling is now same in comment and content section after this patch.Adding screenshot below.
After patch(Firefox browser):
Comment section
Content section
RTBC +1
Comment #41
BhumikaVarshney CreditAttribution: BhumikaVarshney as a volunteer and at OpenSense Labs commentedHi @djsagar ,
Thanks #39 patch resolve the firefox issue.
Comment #42
lauriiiBartik doesn't seem to have any styles that would customize the design of the select element. For that reason, I think it's fine to see different select element designs on different browsers.
Comment #43
lauriiiI was confused by the title/issue summary at first. I read it as if there was a difference in how the select is rendered across different browsers. However, the issue summary explains that it's actually on Firefox but across different instances of the select.
I think we should set the border-radius in a rule targeting .form-select so that it's applied globally on all selects.
Comment #44
alexpottThere's another part of this that affects all browsers - the text box inputs are styled differently too - see the title and subject text input boxes in #40.
Not sure about the best way to resolve this issue.
Comment #49
gumnut CreditAttribution: gumnut at Icon Agency, Australia commentedMoving project to Bartik because its now a contrib module
Comment #50
gumnut CreditAttribution: gumnut at Icon Agency, Australia commentedComment #52
djsagar CreditAttribution: djsagar at OpenSense Labs commentedHi All,
Created MR request for better look, kindly review.
Thanks!