Skip to content

More icons with tooltips for the UI - #1668

Merged
Martii merged 1 commit into
OpenUserJS:masterfrom
Martii:moreIcons
Aug 25, 2019
Merged

Martii merged 1 commit into
OpenUserJS:masterfrom
Martii:moreIcons

Conversation

@Martii

@Martii Martii commented Aug 25, 2019 •

Copy link
Copy Markdown
Member
  • Squeeze in a little more real estate for more the more important things. This is as small as I dare make it and still be readable which in turn adds some vertical white-space occasionally.
  • Fixing a Y2K+100 bug ;)

NOTES:

  • Tested on desktops only... will do more testing after merge. Hopefully everything goes well.
  • Additional squeezing yet to come.

Cumulative changes 6f97ce4...8323bf7


Honorary mention and thanks to @craigbuckler for his fantastic concept article at https://www.sitepoint.com/create-calendar-icon-html5-css3/ for a beautiful layout of a compact date icon. (P.S. not on twitter... hope this will suffice. 😄 )

* Squeeze in a little more real estate for more the more important things. This is as small as I dare make it and still be readable which in turn adds some vertical white-space occasionally.
* Fixing a Y2K+100 bug ;)

NOTES:
* Tested on desktops only... will do more testing after merge. Hopefully everything goes well.
* Additional squeezing yet to come.
@Martii Martii added bug You've guessed it... this means a bug is reported. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. UI Pertains inclusively to the User Interface. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. labels Aug 25, 2019
@Martii
Martii merged commit c68f4cf into OpenUserJS:master Aug 25, 2019
@Martii
Martii deleted the moreIcons branch August 25, 2019 00:51
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Aug 25, 2019
* Chromium based browsers don't like the `<br>` for iconic... that's an unfortunate glitch. Last minute change not tested on desktop. Should have worked since Fx likes it just fine.

NOTE:
* Probably needs hard refresh your window/tab.

Post OpenUserJS#1668
Martii added a commit that referenced this pull request Aug 25, 2019
* Chromium based browsers don't like the `<br>` for iconic... that's an unfortunate glitch. Last minute change not tested on desktop. Should have worked since Fx likes it just fine.

NOTE:
* Probably needs hard refresh your window/tab.

Post #1668

Auto-merge
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Aug 25, 2019
* Linkify as well
* Shrink iconic one more notch... barely readable

Post OpenUserJS#1668
@Martii Martii mentioned this pull request Aug 25, 2019
Martii added a commit that referenced this pull request Aug 25, 2019
* Linkify as well
* Shrink iconic one more notch... barely readable

Post #1668

Auto-merge
@Martii

Martii commented Aug 25, 2019

Copy link
Copy Markdown
Member Author

Btw this should be making room for labeling issues at some point. As I mentioned already "iconic" is about as small as it's going to get without compromising readability for the time being.

Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Aug 25, 2019
* Attempt to focus on unanswered discussions a bit more. dev has a few within a few seconds so this might be futile. Will probe pro in a while. Can check reply count as well if too far off.

Post OpenUserJS#1668
@Martii Martii mentioned this pull request Aug 25, 2019
Martii added a commit that referenced this pull request Aug 25, 2019
* Attempt to focus on unanswered discussions a bit more. dev has a few within a few seconds so this might be futile. Will probe pro in a while. Can check reply count as well if too far off.

Post #1668

Auto-merge
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Aug 25, 2019
* Precision is mostly accurate however there a few 1 second delays.

Post OpenUserJS#1668
@Martii Martii mentioned this pull request Aug 25, 2019
Martii added a commit that referenced this pull request Aug 25, 2019
* Precision is mostly accurate however there a few 1 second delays.

Post #1668

Auto-merge
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Aug 26, 2019
* Using backported class from *bootstrap*@4.x

NOTE:
* Will cram more in later ;)

Post OpenUserJS#1668
Martii added a commit that referenced this pull request Aug 26, 2019
* Using backported class from *bootstrap*@4.x

NOTE:
* Will cram more in later ;)

Post #1668

Auto-merge
Martii added a commit to Martii/OpenUserJS.org that referenced this pull request Sep 6, 2020
* Indicate which ones are defaults and which one is "technically" clear
* Bug fix... so far we don't have a "rating" on Discussions... removed invalid default query value and put in place `created` since we turned off `updated` if not changed. Post OpenUserJS#1668

Post OpenUserJS#703 OpenUserJS#445 OpenUserJS#435
Martii added a commit that referenced this pull request Sep 6, 2020
* Indicate which ones are defaults and which one is "technically" clear
* Bug fix... so far we don't have a "rating" on Discussions... removed invalid default query value and put in place `created` since we turned off `updated` if not changed. Post #1668

Post #703 #445 #435

Auto-merge
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug You've guessed it... this means a bug is reported. CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. UI Pertains inclusively to the User Interface.

Development

Successfully merging this pull request may close these issues.

1 participant