Skip to content

Style.css: Added support for a vertical bar#67

Merged
jdan merged 2 commits intojdan:masterfrom
jacob-macleod:patch-1
Apr 27, 2020
Merged

Style.css: Added support for a vertical bar#67
jdan merged 2 commits intojdan:masterfrom
jacob-macleod:patch-1

Conversation

@jacob-macleod
Copy link
Copy Markdown
Contributor

A vertical bar is an element that was used quite a lot in windows 98. This code makes it easy to do so...

A vertical bar is an element that was used quite a lot in windows 98.
@jdan
Copy link
Copy Markdown
Owner

jdan commented Apr 26, 2020

Hey! Wanna drop a screenshot?

@jacob-macleod
Copy link
Copy Markdown
Contributor Author

Yep:
Screenshot 2020-04-26 at 10 05 44 PM

This is an example of how I used this element in a separate, personal project.

Comment thread style.css Outdated
Comment thread style.css Outdated
Copy link
Copy Markdown
Owner

@jdan jdan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some Qs inline

@jacob-macleod
Copy link
Copy Markdown
Contributor Author

Yes, I have changed both those things and updated the pull request.

@jdan
Copy link
Copy Markdown
Owner

jdan commented Apr 27, 2020

Great work, thank you!

I'll merge this as-is and we'll see if it gets any usage :) Is this element used solely in menu bars? If we ever build one of those we'll have to add it the docs.

@jdan jdan merged commit 64b56d0 into jdan:master Apr 27, 2020
@jacob-macleod
Copy link
Copy Markdown
Contributor Author

Looking at various images, it looks like this element is also used on menus in programs like internet explorer (see image), to separate out different sections of the design
Screenshot 2020-04-27 at 4 47 03 PM

@jacob-macleod jacob-macleod deleted the patch-1 branch April 27, 2020 17:13
@veleek
Copy link
Copy Markdown

veleek commented Apr 30, 2020

@jdan - for draggable/organizable menus this bar is used as a drag handle.

Here's the equivalent in the current version of VS:
image

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants