Merge lp:~diegosarmentero/ubuntuone-control-panel/x-button into lp:ubuntuone-control-panel

Proposed by Diego Sarmentero
Status: Merged
Approved by: Diego Sarmentero
Approved revision: 377
Merged at revision: 377
Proposed branch: lp:~diegosarmentero/ubuntuone-control-panel/x-button
Merge into: lp:ubuntuone-control-panel
Diff against target: 47 lines (+2/-10)
2 files modified
ubuntuone/controlpanel/gui/qt/share_links.py (+2/-5)
ubuntuone/controlpanel/gui/qt/tests/test_share_links.py (+0/-5)
To merge this branch: bzr merge lp:~diegosarmentero/ubuntuone-control-panel/x-button
Reviewer Review Type Date Requested Status
Michał Karnicki (community) Approve
dobey (community) Approve
Review via email: mp+132729@code.launchpad.net

Commit message

- Remove the text on x button pressed (LP: #1070917).

Description of the change

This wasn't working because the focusOutEvent was being fired before the button can detect the click, so the click over the button was actually never being executed.

To post a comment you must log in.
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Michał Karnicki (karni) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'ubuntuone/controlpanel/gui/qt/share_links.py'
--- ubuntuone/controlpanel/gui/qt/share_links.py 2012-11-01 17:57:13 +0000
+++ ubuntuone/controlpanel/gui/qt/share_links.py 2012-11-02 17:19:19 +0000
@@ -84,8 +84,6 @@
84 self._enhanced_line = EnhancedLineEdit(self.ui.line_search,84 self._enhanced_line = EnhancedLineEdit(self.ui.line_search,
85 self._line_close_btn, icon=":/delete_search.png",85 self._line_close_btn, icon=":/delete_search.png",
86 style='enhanced_borderless')86 style='enhanced_borderless')
87 self.ui.line_search.popup.popupHidden.connect(
88 self._hide_line_btn_close_hide)
89 self.ui.line_search.popup.popupShown.connect(87 self.ui.line_search.popup.popupShown.connect(
90 self._hide_line_btn_close_show)88 self._hide_line_btn_close_show)
91 EnhancedLineEdit(self.ui.line_copy_link, self._copy_link_from_line,89 EnhancedLineEdit(self.ui.line_copy_link, self._copy_link_from_line,
@@ -95,6 +93,8 @@
95 self.ui.back_to_file_list.setCursor(QtCore.Qt.PointingHandCursor)93 self.ui.back_to_file_list.setCursor(QtCore.Qt.PointingHandCursor)
96 self.ui.back_to_file_list_2.setCursor(QtCore.Qt.PointingHandCursor)94 self.ui.back_to_file_list_2.setCursor(QtCore.Qt.PointingHandCursor)
97 self.ui.open_in_browser.setCursor(QtCore.Qt.PointingHandCursor)95 self.ui.open_in_browser.setCursor(QtCore.Qt.PointingHandCursor)
96 self._enhanced_line.btn_operation.setCursor(
97 QtCore.Qt.ArrowCursor)
9898
99 self.ui.line_search.itemSelected.connect(self.share_file)99 self.ui.line_search.itemSelected.connect(self.share_file)
100 self.ui.line_search.filesFound.connect(self._load_search_results)100 self.ui.line_search.filesFound.connect(self._load_search_results)
@@ -248,9 +248,6 @@
248 self.ui.line_search.setText('')248 self.ui.line_search.setText('')
249 self.ui.line_search.popup.hide()249 self.ui.line_search.popup.hide()
250 self.ui.line_search.setFocus()250 self.ui.line_search.setFocus()
251
252 def _hide_line_btn_close_hide(self):
253 """Hide the button inside the search line edit-"""
254 self._enhanced_line.btn_operation.hide()251 self._enhanced_line.btn_operation.hide()
255252
256 def _hide_line_btn_close_show(self):253 def _hide_line_btn_close_show(self):
257254
=== modified file 'ubuntuone/controlpanel/gui/qt/tests/test_share_links.py'
--- ubuntuone/controlpanel/gui/qt/tests/test_share_links.py 2012-10-29 13:34:33 +0000
+++ ubuntuone/controlpanel/gui/qt/tests/test_share_links.py 2012-11-02 17:19:19 +0000
@@ -171,11 +171,6 @@
171 self.addCleanup(self.ui.ui.line_search.popup.hide)171 self.addCleanup(self.ui.ui.line_search.popup.hide)
172 self.ui._line_close_btn()172 self.ui._line_close_btn()
173 self.assertFalse(self.ui.ui.line_search.popup.isVisible())173 self.assertFalse(self.ui.ui.line_search.popup.isVisible())
174
175 def test_hide_line_btn_close_hide(self):
176 """Check the state of the inline button."""
177 self.ui._enhanced_line.btn_operation.show()
178 self.ui.ui.line_search.popup.popupHidden.emit()
179 self.assertFalse(self.ui._enhanced_line.btn_operation.isVisible())174 self.assertFalse(self.ui._enhanced_line.btn_operation.isVisible())
180175
181 def test_hide_line_btn_close_show(self):176 def test_hide_line_btn_close_show(self):

Subscribers

People subscribed via source and target branches