Page MenuHomePhorge

Ponder Answer Comments: indent textarea as Comments
ClosedPublic

Authored by valerio.bozzolan on May 4 2023, 21:34.
Tags
None
Referenced Files
F2945557: D25192.1738330520.diff
Thu, Jan 30, 13:35
F2945089: D25192.1738288398.diff
Thu, Jan 30, 01:53
F2941208: D25192.1738101944.diff
Mon, Jan 27, 22:05
F2936840: D25192.1737941266.diff
Sun, Jan 26, 01:27
F2934807: D25192.1737804000.diff
Fri, Jan 24, 11:20
F2931290: D25192.1737702534.diff
Thu, Jan 23, 07:08
F2931207: D25192.1737696843.diff
Thu, Jan 23, 05:34
F2931206: D25192.1737696841.diff
Thu, Jan 23, 05:34

Details

Summary

In a Ponder Answer, when adding a Comment, the textarea indentation was a
a little bit unintuitive.

After this change, the textarea is aligned as a Comment:

BeforeAfter
Ponder - Before.png (694×626 px, 45 KB)
Ponder - After.png (694×626 px, 48 KB)

Other changes are welcome but this seems to me a good minimum.

I also hope this could help not to confuse the Comment field with the Answer field.

Closes T15350

Test Plan
  • create a Ponder Question
  • create an Answer
  • look at the Comments input field that now should be more nicely indented
  • the normal Response field is not changed at all
  • the mobile view is not changed at all
  • click on "Pin Form On Screen" and it must be as before

Diff Detail

Repository
rP Phorge
Branch
ponder-answer-indent
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 361
Build 361: arc lint + arc unit

Event Timeline

webroot/rsrc/css/application/ponder/ponder-view.css
56

Note: I used the .phui-box.phui-comment-form-view selector instead of just .phui-comment-form-view, so that it's more specific, and it overrides the precedent value.

This was my elegant workaround to avoid a CSS !important otherwise necessary.

This revision is now accepted and ready to land.Jul 17 2023, 06:48
webroot/rsrc/css/application/ponder/ponder-view.css
56

↑ My inline comment is not relevant anymore, since we have the :not() selector that is already incidentally more-precise.

So there is no need to have margin-left: 94px; !important.