?
Referenced as Pull Request for: # 6022
avatar rgmears
rgmears
3 Feb 2015

Change ToolTips as noted:

screen shot 2015-02-03 at 08 35 54

Not a bug; just a recommended style change.

avatar rgmears rgmears - open - 3 Feb 2015
avatar rgmears
rgmears - comment - 3 Feb 2015

Regardless the content, the text size should be increased. Or the colours switched. Tiny white text on a black background is difficult to read.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5957.
avatar zero-24
zero-24 - comment - 3 Feb 2015

@rgmears can you send a PR for this change?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5957.
avatar zero-24 zero-24 - change - 3 Feb 2015
Category Administration UI/UX
avatar zero-24 zero-24 - change - 3 Feb 2015
Title
ToolTips
Change how tooltips look
Build 3 staging
avatar brianteeman
brianteeman - comment - 3 Feb 2015

He is planning to - just trying to work out the way to do it with LESS etc

On 3 February 2015 at 16:08, zero-24 notifications@github.com wrote:

@rgmears https://github.com/rgmears can you send a PR for this change?

This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/5957
http://issues.joomla.org/tracker/joomla-cms/5957.


Reply to this email directly or view it on GitHub
#5957 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar losedk
losedk - comment - 3 Feb 2015

Perhaps we could use the Bootstrap popover instead of the tooltip, as shown in #5871

avatar rgmears
rgmears - comment - 3 Feb 2015

Perhaps we could use the Bootstrap popover instead of the tooltip

That would work. My altered styling is similar to the popover. Contrary to what Brian said in the user-interface group I think the only file that needs changing is the tooltip.less (in the template folder) which, once compiled, will change the template.css. This would have to be done for both ISIS and HATHOR. Is that correct?

avatar rgmears
rgmears - comment - 3 Feb 2015

Except the a popover places content to the right while a tooltip places content above.

avatar losedk
losedk - comment - 3 Feb 2015

skaermbillede 2015-02-03 kl 18 36 03

avatar dgt41
dgt41 - comment - 3 Feb 2015

@rgmears @losedk please don’t change the tooltips with popovers. Also I would like to have tooltips with different style than popovers

avatar losedk
losedk - comment - 3 Feb 2015

@dgt41 I'm not changing anything :) I just think it's better to use the std. Bootstrap features, if they were to change.

avatar rgmears
rgmears - comment - 3 Feb 2015

@dgt41 How would you like the tooltip to be different. Is my original suggestion (at the top) acceptable. My intention was to replace the tiny white text on black -- which is difficult to read -- with something more legible.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5957.
avatar brianteeman brianteeman - change - 4 Feb 2015
Labels Added: ?
avatar rgmears
rgmears - comment - 4 Feb 2015

How do I upload LESS and CSS files here?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5957.
avatar Bakual
Bakual - comment - 5 Feb 2015

You edit the LESS file and run https://github.com/joomla/joomla-cms/blob/staging/build/generatecss.php afterwards. That will regenerate the CSS files.

avatar rgmears
rgmears - comment - 5 Feb 2015

@Bakual
Created two pull requests: isis/less/template.less and isis/less/variables.less
Not sure what to do next.

avatar Bakual
Bakual - comment - 5 Feb 2015

If you only use the GitHub UI, then you will not be able to run it.
If you work with a local branch, you run the generatecss.php file using php generatecss.php (assuming PHP is set up to run from CLI like this).

Also it would be great if you could create one PR instead of the two. With the GitHub UI, you can go to one PR and see what branch it originates from, and the select that branch in your repo and navigate to the file you want to change. Edit that file while still having that branch active and it will automagically update your PR with the changes.

avatar rgmears
rgmears - comment - 5 Feb 2015

I am changing three files: two LESS files and then the generated CSS file. How do I do that in one pull request. And I do not know what you mean by running the generatecss.php file. How? I do not know how to use Command Line. Sorry. (Designer not programmer)


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5957.
avatar Bakual
Bakual - comment - 5 Feb 2015

you will not be able to do that using the GitHub UI since you would need to run it locally from command line and then push it back using Git.
You can try to apply the changes manually, I think being a designer you know better than me how to create a CSS file from LESS. Just make sure you make your edits in the existing branch in your repo.
Follow that link to get to the patch-1 branch of your own fork: https://github.com/rgmears/joomla-cms/tree/patch-1
Everything you change in that branch, automatically becomes part of your PR #5982. So just edit the other two files within that branch.

avatar rgmears
rgmears - comment - 5 Feb 2015

I created three PRs, one for each file. The template.css file was (as were the others) copied from my local installation. I use a compiler called WinLess. (It's not the best but it works.) For whatever reason the template.css file did not get saved by GitHub. However, when I first saved it it did not record changes correctly.

There are now three patch files: rgmears-patch-1, patch-1, and patch-2. It would appear I should delete two of them. The first one — rgmears-patch-1 — appears to be the one you have linked to PR #5982. Is that correct?

avatar rgmears
rgmears - comment - 5 Feb 2015

Apologies. The template.css file was saved by GitHub. it is in rgmears-patch-1. How do I move the other two in to that PR?

avatar rgmears
rgmears - comment - 6 Feb 2015

Duh. Thanks Thomas. It's done.

Shall I delete the other two: patch-1 and patch-2 ?

avatar Bakual
Bakual - comment - 6 Feb 2015

Yes, please close the PRs and afterwards you can delete the branches.

avatar rgmears
rgmears - comment - 6 Feb 2015

Done.

avatar rgmears
rgmears - comment - 7 Feb 2015

How do we add the Pull Request to this thread?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5957.
avatar infograf768
infograf768 - comment - 7 Feb 2015

You don't. Just create a new PR and cross link to this Issue.

avatar rgmears
rgmears - comment - 7 Feb 2015

So this doesn't count?

(rgmears#1)


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5957.
avatar rgmears
rgmears - comment - 8 Feb 2015
avatar brianteeman brianteeman - change - 9 Feb 2015
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-02-09 00:05:20
Closed_By brianteeman
avatar brianteeman brianteeman - close - 9 Feb 2015

Add a Comment

Login with GitHub to post a comment