Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comment layers are not visible enough #3195

Closed
therealwolf42 opened this issue Jan 27, 2019 · 3 comments · Fixed by #3394
Closed

Comment layers are not visible enough #3195

therealwolf42 opened this issue Jan 27, 2019 · 3 comments · Fixed by #3394

Comments

@therealwolf42
Copy link

therealwolf42 commented Jan 27, 2019

Currently, we only have this little dot showing us on which comment-level/layer we are, which is not visible enough IMO. I actually didn't know it existed, because I didn't see it until @eonwarped told me about it:

image

I would prefer to have something along the lines of this:

image

@roadscape
Copy link
Contributor

Making comments a little more condensed and as well-defined blocks would also help

@quochuy
Copy link
Contributor

quochuy commented May 5, 2019

This feature had been added to my Chrome Extension Steemed Phish for a while now. It was achieved by injecting this CSS into Steemit pages:
.Comment__body { border-left: 3px dotted #ededed; padding-left: 5px; padding-bottom: 2px; }

@roadscape well-defined blocks don't resolve the issue this feature of Steemed Phish was created for which is some users were posting a comment containing an image that was faking other comments underneath leading to users clicking on it to go to a phishing site.

Screen Shot 2019-05-06 at 8 26 02 am

quochuy pushed a commit to quochuy/condenser that referenced this issue May 10, 2019
…lock (header, body, footer) this improves visibility of each comment layer.
@quochuy
Copy link
Contributor

quochuy commented May 10, 2019

#3332

Screen Shot 2019-05-10 at 9 58 15 am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants