Skip to content
This repository has been archived by the owner on May 3, 2022. It is now read-only.

Fix EZP-27330: Full name in preview box and on hover of the link in UDW #873

Conversation

adamwojs
Copy link
Member

@adamwojs adamwojs commented Jun 2, 2017

JIRA: https://jira.ez.no/browse/EZP-27330

Description

This PR fixes the preview box so it can show the full name of selected content. Based on CSS proposed by @inakijv in JIRA issue.

Screenshot

zrzut ekranu 2017-06-02 o 13 40 07 2

Related PRs

Copy link

@SylvainGuittard SylvainGuittard left a comment

Choose a reason for hiding this comment

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

Thanks @adamwojs

Copy link
Contributor

@andrerom andrerom left a comment

Choose a reason for hiding this comment

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

+1, @dpobel / @StephaneDiot: looks ok?

@StephaneDiot
Copy link
Contributor

StephaneDiot commented Jun 5, 2017

+1 for the CSS changes of the PR but it's not corresponding to the description: this PR doesn't fixes the preview box so it can show the full name of selected content

For that you need to apply the CSS changes that is in #863 on Resources/public/css/views/universaldiscovery/selected.css (removing text-overflow: ellipsis, overflow: hidden and white-space: nowrap ...)

font-weight: bold;
margin-top: 5px;
Copy link
Contributor

Choose a reason for hiding this comment

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

please no px, we use relative unit, em usually for margins

}

.ez-view-universaldiscoveryselectedview .ez-ud-selected-infos .ez-ud-selected-info {
margin-bottom: 0.5em;
margin-left: 0;
font-size: 0.8em;
Copy link
Contributor

Choose a reason for hiding this comment

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

to not repeat that, what about defining the font-size on .ez-ud-selected-infos ?

@adamwojs
Copy link
Member Author

adamwojs commented Jun 5, 2017

@dpobel @StephaneDiot Thank you for review. I've applied suggested changes.

Copy link
Contributor

@dpobel dpobel left a comment

Choose a reason for hiding this comment

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

actually the patch should be made for the 1.9 branch

@adamwojs adamwojs changed the base branch from master to 1.9 June 12, 2017 09:21
@adamwojs adamwojs force-pushed the Fix-EZP-27330_full_name_in_preview_box_and_on_hover_of_the_link branch from e48af14 to 69a05ce Compare June 12, 2017 09:30
@andrerom andrerom merged commit 0a7c70e into ezsystems:1.9 Jun 12, 2017
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

5 participants